Skip to content

refactor: tsconfig tooling cleanup and updates#607

Draft
hazre wants to merge 12 commits intodevfrom
hazre/refactor/tsconfig-tooling-cleanup
Draft

refactor: tsconfig tooling cleanup and updates#607
hazre wants to merge 12 commits intodevfrom
hazre/refactor/tsconfig-tooling-cleanup

Conversation

@hazre
Copy link
Copy Markdown
Member

@hazre hazre commented Mar 31, 2026

Description

This also incorporates changes from #517

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

AI disclosure:

  • Partially AI assisted (clarify which code was AI assisted and briefly explain what it does).
  • Fully AI generated (explain what all the generated code does in moderate detail).

hazre and others added 11 commits March 28, 2026 17:25
Apply code quality enforcement across ESLint, storage guards, coverage checks, and contributor docs.

Co-authored-by: Evie Gauthier <evie@gauthier.id>
…g-tooling-cleanup

# Conflicts:
#	package.json
#	src/app/components/Modal500.tsx
#	src/app/components/RoomNotificationSwitcher.tsx
#	src/app/components/image-viewer/ImageViewer.tsx
#	src/app/components/room-avatar/RoomAvatar.tsx
#	src/app/components/setting-tile/SettingTile.tsx
#	src/app/components/text-viewer/TextViewer.tsx
#	src/app/components/url-preview/ClientPreview.tsx
#	src/app/components/user-avatar/UserAvatar.tsx
#	src/app/features/message-search/SearchResultGroup.tsx
#	src/app/features/room/message/MessageEditor.tsx
#	src/app/features/settings/cosmetics/Themes.tsx
#	src/app/features/settings/developer-tools/SentrySettings.tsx
#	src/app/features/settings/emojis-stickers/EmojisStickers.tsx
#	src/app/features/settings/notifications/AllMessages.tsx
#	src/app/features/settings/notifications/KeywordMessages.tsx
#	src/app/features/settings/notifications/NotificationModeSwitcher.tsx
#	src/app/features/settings/notifications/SpecialMessages.tsx
#	src/app/hooks/useImageGestures.ts
#	src/app/hooks/useMentionClickHandler.ts
#	src/app/hooks/usePerMessageProfile.ts
#	src/app/plugins/react-custom-html-parser.tsx
#	src/app/plugins/react-prism/ReactPrism.tsx
#	src/app/utils/sanitize.ts
#	src/client/slidingSync.ts
@github-actions

This comment was marked as spam.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

OpenTofu plan for production

Plan: 2 to add, 0 to change, 2 to destroy.
OpenTofu used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
-/+ destroy and then create replacement

OpenTofu will perform the following actions:

  # cloudflare_worker_version.site must be replaced
-/+ resource "cloudflare_worker_version" "site" {
!~      annotations         = {
+           workers_message      = (known after apply)
+           workers_tag          = (known after apply)
!~          workers_triggered_by = "create_version_api" -> (known after apply)
        } -> (known after apply)
!~      assets              = { # forces replacement
!~          asset_manifest_sha256 = "a3409128ca2f72dcfa16dbeee4fe5b54979eacc5040f0557790adc1d43cc3412" -> "0611c9d79fd1f50c4475346796638cef733ce76bc62fb14f5c324bab054456f2" # forces replacement
!~          directory             = "/home/runner/work/Sable/Sable/dist" -> "/github/workspace/dist"
#            (1 unchanged attribute hidden)
        }
+       compatibility_flags = (known after apply)
!~      created_on          = "2026-03-30T03:59:23Z" -> (known after apply)
!~      id                  = "************************************" -> (known after apply)
+       limits              = (known after apply)
+       main_script_base64  = (known after apply)
!~      number              = 627 -> (known after apply)
!~      source              = "terraform" -> (known after apply)
+       startup_time_ms     = (known after apply)
#        (4 unchanged attributes hidden)
    }

  # cloudflare_workers_deployment.site must be replaced
-/+ resource "cloudflare_workers_deployment" "site" {
!~      annotations  = { # forces replacement
!~          workers_message      = "v1.13.1" -> "refactor: tsconfig tooling cleanup and updates"
!~          workers_triggered_by = "deployment" -> (known after apply)
        }
!~      author_email = "cloudflare@sl.sable.moe" -> (known after apply)
!~      created_on   = "2026-03-30T03:59:24Z" -> (known after apply)
!~      id           = "************************************" -> (known after apply)
!~      source       = "terraform" -> (known after apply)
!~      versions     = [ # forces replacement
!~          {
!~              version_id = "************************************" -> (known after apply)
#                (1 unchanged attribute hidden)
            },
        ]
#        (3 unchanged attributes hidden)
    }

Plan: 2 to add, 0 to change, 2 to destroy.

Warning: Attribute Deprecated

  with cloudflare_workers_custom_domain.site,
  on main.tf line 41, in resource "cloudflare_workers_custom_domain" "site":
  41:   environment = "production"

This attribute is deprecated.

(and one more similar warning elsewhere)

📝 Plan generated in Cloudflare Infra #129

@hazre hazre added the internal label Apr 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

Status Preview URL Commit Alias Updated (UTC)
✅ Deployment successful! https://pr-607-sable.raspy-dream-bb1d.workers.dev 800044a pr-607 Thu, 02 Apr 2026 12:07:59 GMT

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant