typescript strict null checks#1816
Merged
Merged
Conversation
|
Important Review skippedToo many files! This PR contains 259 files, which is 109 over the limit of 150. To get a review, narrow the scope: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (259)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR enables TypeScript strictNullChecks for the backend and updates a broad set of backend code (entities/DTOs/use-cases/guards/utilities) to be explicit about null | undefined, adding defensive checks and adjusting return/parameter types to satisfy stricter type safety.
Changes:
- Enable
strictNullChecksin backend TypeScript compilation. - Update many backend flows to handle nullable values explicitly (guards, repositories/use-cases, DTOs/entities, helpers).
- Introduce user-info response-object types/utilities for SaaS microservice and tighten several API return types.
Reviewed changes
Copilot reviewed 259 out of 259 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| backend/tsconfig.src.json | Enable strictNullChecks. |
| backend/src/shared/services/turnstile.service.ts | Add Turnstile secret-key guard. |
| backend/src/shared/database/database.providers.ts | Make DataSource cache nullable. |
| backend/src/shared/config/app-config.ts | Make parsed DB URL fields nullable/optional. |
| backend/src/selfhosted-operations/application/use-cases/create-initial-user.use.case.ts | Handle possibly-null user info DS. |
| backend/src/microservices/saas-microservice/utils/build-found-user-info-ro.ts | New builders for “found user info” ROs. |
| backend/src/microservices/saas-microservice/use-cases/saas-usual-register-user.use.case.ts | Null/undefined adjustments in registration response. |
| backend/src/microservices/saas-microservice/use-cases/saas-use-cases.interface.ts | Return RO types instead of entities. |
| backend/src/microservices/saas-microservice/use-cases/register-demo-user-account.use.case.ts | Null/undefined adjustments in demo registration. |
| backend/src/microservices/saas-microservice/use-cases/login-with-google.use.case.ts | Nullable repository return + audit payload tweaks. |
| backend/src/microservices/saas-microservice/use-cases/login-with-github.use.case.ts | Audit payload tweaks for nullable userId. |
| backend/src/microservices/saas-microservice/use-cases/get-users-infos-by-email.use.case.ts | Return sanitized RO list instead of entities. |
| backend/src/microservices/saas-microservice/use-cases/get-user-info.use.case.ts | Return sanitized RO instead of entity mutation. |
| backend/src/microservices/saas-microservice/use-cases/delete-connection-for-hosted-db.use.case.ts | Null handling + formatting. |
| backend/src/microservices/saas-microservice/use-cases/create-connection-for-hosted-db.use.case.ts | Null guards; safer updates; avoid deleting from typed entity directly. |
| backend/src/microservices/saas-microservice/saas.controller.ts | Update controller return types to ROs. |
| backend/src/microservices/saas-microservice/data-structures/hosted-connection-credentials.ro.ts | Make fields optional/nullable. |
| backend/src/microservices/saas-microservice/data-structures/found-user-info.ro.ts | New RO types based on UserEntity data keys. |
| backend/src/microservices/saas-microservice/data-structures/found-connection-info.ro.ts | Make fields optional/nullable. |
| backend/src/microservices/gateways/saas-gateway.ts/saas-company-gateway.service.ts | Null checks on gateway responses. |
| backend/src/microservices/gateways/saas-gateway.ts/base-saas-gateway.service.ts | Allow null request body. |
| backend/src/interceptors/sentry.interceptor.ts | Ensure interceptor returns next.handle() on error. |
| backend/src/helpers/validators/validation-helper.ts | Fix JSON validation return; remove .at() nullability issue. |
| backend/src/helpers/operate-values-between-curlies.ts | RegExp exec nullable; replacement guards. |
| backend/src/helpers/is-connection-entity-agent.ts | Allow null/undefined connection type param. |
| backend/src/helpers/encryption/encryptor.ts | Add config checks + overloads for nullable inputs. |
| backend/src/helpers/constants/constants.ts | Narrow filter types; allow null return from DSN helper. |
| backend/src/helpers/check-field-autoincrement.ts | Accept nullable args. |
| backend/src/guards/tables-receive.guard.ts | Add missing-user guard; make params optional. |
| backend/src/guards/table-read.guard.ts | Add missing-user guard; make params optional. |
| backend/src/guards/table-edit.guard.ts | Add missing-user guard; make params optional. |
| backend/src/guards/table-delete.guard.ts | Add missing-user guard; make params optional. |
| backend/src/guards/table-ai-request.guard.ts | Add missing-user guard; make params optional. |
| backend/src/guards/table-add.guard.ts | Add missing-user guard; make params optional. |
| backend/src/guards/schema-change-ownership.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/schema-change-batch-ownership.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/panel-read.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/panel-edit.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/paid-feature.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/group-read.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/group-edit.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/dashboard-read.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/dashboard-edit.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/dashboard-create.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/connection-read.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/connection-edit.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/connection-diagram.guard.ts | Guard missing user; nullable params. |
| backend/src/guards/company-user.guard.ts | Handle missing userId. |
| backend/src/guards/company-admin.guard.ts | Handle missing userId. |
| backend/src/guards/action-event-trigger.guard.ts | Guard missing user; nullable params. |
| backend/src/exceptions/utils/processing-messages-replace.ts | Optional-chain match result. |
| backend/src/exceptions/text/messages.ts | Add missing message constant. |
| backend/src/exceptions/custom-exceptions/validation-exception.ts | Handle missing constraints object. |
| backend/src/enums/widget-type.enum.ts | Add Money widget type. |
| backend/src/entities/widget/utils/validate-create-widgets-ds.ts | Guard missing widget params; fix message param. |
| backend/src/entities/widget/use-cases/find-table-widgets.use.case.ts | Handle missing connection; avoid null email param. |
| backend/src/entities/widget/use-cases/create-update-delete-table-widgets.use.case.ts | Handle missing connection; avoid null email param. |
| backend/src/entities/widget/table-widget.interface.ts | Make widget_params nullable. |
| backend/src/entities/widget/table-widget.entity.ts | Make widget JSON fields nullable. |
| backend/src/entities/widget/application/data-sctructures/found-table-widgets.ds.ts | Nullable widget params/options. |
| backend/src/entities/widget/application/data-sctructures/create-table-widgets.ds.ts | Nullable widget params/options; optional type. |
| backend/src/entities/visualizations/panel/use-cases/test-db-query.use.case.ts | Default nullable strings before DAO call. |
| backend/src/entities/visualizations/panel/use-cases/execute-panel.use.case.ts | Default nullable strings before DAO call. |
| backend/src/entities/visualizations/panel-position/use-cases/generate-table-dashboard-with-ai.use.case.ts | Default userEmail + explain text. |
| backend/src/entities/visualizations/panel-position/use-cases/generate-panel-position-with-ai.use.case.ts | Default userEmail + explain text. |
| backend/src/entities/user/utils/is-jwt-scope-need.util.ts | Normalize scope to empty array; nullable company. |
| backend/src/entities/user/utils/generate-gwt-token.ts | Throw on missing JWT secrets; omit empty scopes. |
| backend/src/entities/user/utils/build-created-user.ds.ts | Allow null return when user missing. |
| backend/src/entities/user/utils/build-connection-entities-from-test-dtos.ts | Default missing boolean flag. |
| backend/src/entities/user/user.entity.ts | Make several columns explicitly nullable. |
| backend/src/entities/user/user.controller.ts | Return SaveUserSettingsDs from settings endpoints. |
| backend/src/entities/user/user-session-settings/user-session-settings.entity.ts | Nullable userSettings column. |
| backend/src/entities/user/user-session-settings/reposiotory/user-session-settings-repository.interface.ts | Nullable returns/inputs. |
| backend/src/entities/user/user-session-settings/reposiotory/user-session-settings-custom-repository.extension.ts | Nullable returns/inputs. |
| backend/src/entities/user/user-invitation/user-invitation.entity.ts | Nullable ownerId. |
| backend/src/entities/user/user-helper.service.ts | Normalize null vs undefined fields. |
| backend/src/entities/user/use-cases/verify-user-email.use.case.ts | Guard missing user. |
| backend/src/entities/user/use-cases/usual-login-use.case.ts | Nullable user + optional audit params. |
| backend/src/entities/user/use-cases/request-reset-user-password.use.case.ts | Optional-chain mailing result. |
| backend/src/entities/user/use-cases/request-change-user-email.use.case.ts | Guard missing user; optional-chain mailing result. |
| backend/src/entities/user/use-cases/otp-login-use.case.ts | Guard missing OTP secret; optional audit params. |
| backend/src/entities/user/repository/user.repository.interface.ts | Nullable returns for lookups. |
| backend/src/entities/user/repository/user-custom-repository-extension.ts | Nullable params/returns; avoid deleting relations directly. |
| backend/src/entities/user/dto/found-user.dto.ts | Nullable/optional DTO fields. |
| backend/src/entities/user/application/data-structures/usual-login.ds.ts | Nullable gclid. |
| backend/src/entities/user/application/data-structures/save-user-settings.ds.ts | Nullable userSettings. |
| backend/src/entities/user/application/data-structures/registered-user.ds.ts | Nullable name/provider. |
| backend/src/entities/user/application/data-structures/register-user-ds.ts | Nullable register inputs. |
| backend/src/entities/user/application/data-structures/found-user-in-group.ds.ts | Nullable name/provider. |
| backend/src/entities/user-sign-in-audit/use-cases/find-sign-in-audit-logs.use.case.ts | Nullable date range vars. |
| backend/src/entities/user-sign-in-audit/sign-in-audit.service.ts | Normalize nullable email. |
| backend/src/entities/user-sign-in-audit/sign-in-audit.entity.ts | Nullable columns + explicit types. |
| backend/src/entities/user-sign-in-audit/repository/sign-in-audit-repository.interface.ts | Nullable date filters. |
| backend/src/entities/user-sign-in-audit/repository/sign-in-audit-custom-repository-extension.ts | Normalize nullable email. |
| backend/src/entities/user-sign-in-audit/dto/found-sign-in-audit-record.ds.ts | Nullable audit DTO fields. |
| backend/src/entities/user-sign-in-audit/dto/create-sign-in-audit-record.ds.ts | Nullable userId. |
| backend/src/entities/user-secret/user-secret.entity.ts | Nullable timestamp/hash fields. |
| backend/src/entities/user-secret/use-cases/update-secret.use.case.ts | Guard missing masterHash. |
| backend/src/entities/user-secret/application/dto/secret-list.dto.ts | Nullable timestamps in DTO. |
| backend/src/entities/user-secret/application/dto/found-secret.dto.ts | Nullable timestamps in DTO. |
| backend/src/entities/user-secret/application/data-structures/update-secret.ds.ts | Nullable timestamps. |
| backend/src/entities/user-secret/application/data-structures/get-secrets.ds.ts | Nullable timestamps. |
| backend/src/entities/user-secret/application/data-structures/found-secret.ds.ts | Nullable timestamps. |
| backend/src/entities/user-secret/application/data-structures/created-secret.ds.ts | Nullable timestamps. |
| backend/src/entities/user-actions/use-cases/check-users-logs-and-update-actions.use.case.ts | Nullable action lookup return. |
| backend/src/entities/user-actions/use-cases/check-users-actions-and-mailing-users.use.case.ts | Guard missing user; nullable action lookup return. |
| backend/src/entities/table/utils/validate-table-row.util.ts | Remove .at() nullability; safer default checks. |
| backend/src/entities/table/utils/validate-connection.util.ts | Normalize agent email to string. |
| backend/src/entities/table/utils/save-tables-info-in-database.util.ts | Default nullable column_default. |
| backend/src/entities/table/utils/save-table-info-in-database-orchestrator.util.ts | Guard missing decrypted connection; avoid undefined email. |
| backend/src/entities/table/utils/process-uuids-in-row-util.ts | Skip when widget params missing. |
| backend/src/entities/table/utils/process-referenced-tables.util.ts | Add type-guard filters; nullable display_name. |
| backend/src/entities/table/utils/hash-passwords-in-row.util.ts | Skip when widget params missing. |
| backend/src/entities/table/utils/form-full-table-structure.ts | Default nullable column_default. |
| backend/src/entities/table/utils/build-common-table-settings-input.util.ts | New utility to normalize nullable settings. |
| backend/src/entities/table/utils/attach-foreign-column-names.util.ts | Avoid optional chaining pitfalls. |
| backend/src/entities/table/use-cases/update-row-in-table.use.case.ts | Use normalized settings input; minor null-safe list_fields. |
| backend/src/entities/table/use-cases/get-table-rows.use.case.ts | Normalize filters/autocomplete/settings inputs. |
| backend/src/entities/table/use-cases/get-row-by-primary-key.use.case.ts | Use normalized settings input. |
| backend/src/entities/table/use-cases/find-tables-in-connection.use.case.ts | Nullable connection + default userEmail + hidden tables normalization. |
| backend/src/entities/table/use-cases/find-tables-in-connection-v2.use.case.ts | Nullable connection + hidden tables normalization. |
| backend/src/entities/table/use-cases/export-csv-from-table.use.case.ts | Normalize filters + settings input. |
| backend/src/entities/table/use-cases/delete-rows-from-table.use.case.ts | Normalize settings input; nullable error field; safe defaults. |
| backend/src/entities/table/use-cases/delete-row-from-table.use.case.ts | Normalize settings input. |
| backend/src/entities/table/use-cases/bulk-update-rows-in-table.use.case.ts | Normalize settings input. |
| backend/src/entities/table/use-cases/add-row-in-table.use.case.ts | Normalize settings input; ensure deterministic failure behavior. |
| backend/src/entities/table/table.controller.ts | Initialize numeric vars; guard missing connection. |
| backend/src/entities/table/table-datastructures.ts | Nullable display_name/identity_column; change widgets type. |
| backend/src/entities/table/application/data-structures/found-table.ds.ts | Nullable display_name/icon. |
| backend/src/entities/table/application/data-structures/found-table-structure.ds.ts | Nullable column_default/length; optional params. |
| backend/src/entities/table/application/data-structures/found-table-rows.ds.ts | Optional ordering fields; nullable identity column. |
| backend/src/entities/table-settings/personal-table-settings/use-cases/create-update-personal-table-settings.use.case.ts | Guard missing connection; avoid null email param. |
| backend/src/entities/table-settings/common-table-settings/utils/build-new-table-settings-entity.ts | Default nullable arrays/fields and booleans. |
| backend/src/entities/table-settings/common-table-settings/use-cases/update-table-settings.use.case.ts | Guard missing connection; normalize masterPwd. |
| backend/src/entities/table-settings/common-table-settings/use-cases/create-table-settings.use.case.ts | Guard missing connection; normalize masterPwd. |
| backend/src/entities/table-settings/common-table-settings/table-settings.entity.ts | Make several columns nullable with explicit types. |
| backend/src/entities/table-settings/common-table-settings/table-settings.controller.ts | Fix DTO type in validator method signature. |
| backend/src/entities/table-settings/application/data-structures/found-table-settings.ds.ts | Nullable display/identity/icon/list_per_page; nullable arrays. |
| backend/src/entities/table-settings/application/data-structures/create-table-settings.ds.ts | Make many fields optional/nullable. |
| backend/src/entities/table-schema/utils/execute-schema-change.ts | Avoid null email param. |
| backend/src/entities/table-schema/use-cases/rollback-schema-change.use-case.ts | Normalize master password. |
| backend/src/entities/table-schema/use-cases/rollback-batch-schema-changes.use-case.ts | Normalize master password. |
| backend/src/entities/table-schema/use-cases/generate-schema-change.use-case.ts | Normalize master password. |
| backend/src/entities/table-schema/use-cases/approve-batch-schema-changes.use-case.ts | Normalize master password. |
| backend/src/entities/table-schema/use-cases/approve-and-apply-schema-change.use-case.ts | Normalize master password. |
| backend/src/entities/table-schema/ai/run-schema-change-ai-loop.ts | Avoid null userEmail. |
| backend/src/entities/table-logs/use-cases/find-logs.use.case.ts | Nullable date range vars. |
| backend/src/entities/table-logs/use-cases/export-logs-as-csv.use.case.ts | Nullable date range vars; import ordering cleanup. |
| backend/src/entities/table-logs/table-logs.service.ts | Return null when audit disabled/missing user. |
| backend/src/entities/table-logs/table-logs.entity.ts | Explicit varchar type for nullable field. |
| backend/src/entities/table-logs/repository/table-logs-repository.interface.ts | Nullable dates/tableName. |
| backend/src/entities/table-logs/application/data-structures/create-log-record.ds.ts | Nullable operation_custom_action_name. |
| backend/src/entities/table-filters/utils/validate-table-filters-data.util.ts | Avoid null email param. |
| backend/src/entities/table-filters/utils/build-created-table-filters-response-object.util.ts | Null-safe dynamic filter object build. |
| backend/src/entities/table-filters/use-cases/update-table-filters.use.case.ts | Guard missing connection. |
| backend/src/entities/table-filters/use-cases/create-table-filters.use.case.ts | Guard missing connection; normalize masterPwd. |
| backend/src/entities/table-filters/application/response-objects/create-table-filters.dto.ts | Nullable dynamic_column. |
| backend/src/entities/table-filters/application/data-structures/create-table-filters.ds.ts | Nullable masterPwd. |
| backend/src/entities/table-categories/use-cases/find-table-categories-with-tables.use.case.ts | Nullable connection + hidden tables normalization + type guards. |
| backend/src/entities/table-categories/use-cases/create-or-update-table-categories.use.case.ts | Guard missing connection. |
| backend/src/entities/table-categories/dto/found-table-categories-with-tables.ro.ts | Nullable category_color. |
| backend/src/entities/table-actions/table-actions-module/utils/build-table-action-with-rule.util.ts | Default null/empty action fields. |
| backend/src/entities/table-actions/table-actions-module/table-action.entity.ts | Nullable URL fields with explicit types. |
| backend/src/entities/table-actions/table-actions-module/table-action-activation.service.ts | Add user/null checks; nullable URL/slack config handling. |
| backend/src/entities/table-actions/table-action-rules-module/use-cases/update-action-rule-with-actions-and-events.use.case.ts | Guard missing action/event; normalize nullable fields. |
| backend/src/entities/table-actions/table-action-rules-module/use-cases/create-action-rule.use.case.ts | Guard missing connection; normalize nullable fields. |
| backend/src/entities/table-actions/table-action-rules-module/use-cases/activate-actions-in-rule.use.case.ts | Guard missing connection; pass explicit CUSTOM trigger op. |
| backend/src/entities/table-actions/table-action-rules-module/application/dto/found-action-rules-with-actions-and-events.dto.ts | Nullable event title. |
| backend/src/entities/table-actions/table-action-rules-module/application/dto/activated-table-actions.dto.ts | Nullable location. |
| backend/src/entities/table-actions/table-action-rules-module/application/data-structures/create-action-rules.ds.ts | Make several fields optional/nullable. |
| backend/src/entities/table-actions/table-action-events-module/utils/build-action-event-with-rule.util.ts | Default nullable icon. |
| backend/src/entities/table-actions/table-action-events-module/action-event.entity.ts | Nullable title/icon with explicit types. |
| backend/src/entities/shared-jobs/shared-jobs.service.ts | Avoid null userEmail; safer widget/settings mutation. |
| backend/src/entities/s3-widget/use-cases/get-s3-upload-url.use.case.ts | Guard missing widget params. |
| backend/src/entities/s3-widget/use-cases/get-s3-file-url.use.case.ts | Guard missing params; normalize settings input + agent email. |
| backend/src/entities/group/utils/build-remove-user-from-group-result.ds.ts | Handle missing users array. |
| backend/src/entities/group/utils/biuld-found-group-response.dto.ts | Filter null user infos. |
| backend/src/entities/group/use-cases/update-group-title.use.case.ts | Guard missing connection/group relation. |
| backend/src/entities/group/use-cases/saas-add-user-in-group-v2.use.case.ts | Guard missing group/users. |
| backend/src/entities/group/use-cases/remove-user-from-group.use.case.ts | Guard missing group/users. |
| backend/src/entities/group/group.entity.ts | Make connection relation optional. |
| backend/src/entities/email/email/email.service.ts | Normalize return types to include null; safer error returns. |
| backend/src/entities/email/email/email.interface.ts | Constrain generic payload type. |
| backend/src/entities/email/email/email.generator.ts | Match constrained generic payload type. |
| backend/src/entities/email/email-config/email-config.service.ts | Default nullable SMTP config parts. |
| backend/src/entities/custom-field/utils/validate-create-custom-field-dto.ts | Avoid null email param. |
| backend/src/entities/custom-field/use-cases/update-custom-field.use.case.ts | Guard missing connection. |
| backend/src/entities/custom-field/use-cases/create-custom-fields.use.case.ts | Guard missing connection. |
| backend/src/entities/connection/utils/validate-create-connection-data.ts | Fix port validation ordering. |
| backend/src/entities/connection/utils/update-connection-entity-for-restoration.ts | Default nullable booleans. |
| backend/src/entities/connection/utils/process-aws-connection.util.ts | Guard missing host. |
| backend/src/entities/connection/utils/is-test-connection-util.ts | Default missing host to empty string. |
| backend/src/entities/connection/utils/is-sql-connection-type.util.ts | Handle null/undefined type. |
| backend/src/entities/connection/utils/is-host-allowed.ts | Default missing host to empty string. |
| backend/src/entities/connection/utils/decrypt-connection-credentials-async.ts | Default nullable booleans. |
| backend/src/entities/connection/utils/build-found-user-group-in-connection-dto.util.ts | Filter null user infos with type guard. |
| backend/src/entities/connection/utils/build-connection-entity.ts | Default nullable booleans. |
| backend/src/entities/connection/use-cases/validate-connection-master-password.use.case.ts | Nullable connection lookup result. |
| backend/src/entities/connection/use-cases/update-connection.use.case.ts | Guard missing connection; null-safe host check. |
| backend/src/entities/connection/use-cases/update-connection-title.use.case.ts | Guard missing connection. |
| backend/src/entities/connection/use-cases/update-connection-master-password.use.case.ts | Guard missing connection. |
| backend/src/entities/connection/use-cases/unfreeze-connection.use.case.ts | Guard missing connection. |
| backend/src/entities/connection/use-cases/test-connection.use.case.ts | Null-safe host checks; null-safe slack message. |
| backend/src/entities/connection/use-cases/restore-connection-use.case.ts | Null-safe host check; guard missing saved connection. |
| backend/src/entities/connection/use-cases/preview-connection-diagram.use.case.ts | Normalize agent email param; avoid null to DAO. |
| backend/src/entities/connection/use-cases/get-user-permissions-for-group-in-connection.use.case.ts | Guard missing connection; minor cleanup. |
| backend/src/entities/connection/use-cases/get-permissions-for-group-in-connection.use.case.ts | Guard missing connection. |
| backend/src/entities/connection/use-cases/get-connection-diagram.use.case.ts | Normalize agent email param; avoid null to DAO. |
| backend/src/entities/connection/use-cases/find-one-connection.use.case.ts | Null-safe master hash verify; nullable connectionProperties. |
| backend/src/entities/connection/use-cases/create-group-in-connection.use.case.ts | Guard missing connection/user. |
| backend/src/entities/connection/use-cases/create-connection.use.case.ts | Nullable author; null-safe company update; nullable token. |
| backend/src/entities/connection/connection.entity.ts | Make many fields optional/nullable with explicit types. |
| backend/src/entities/connection/connection.controller.ts | Nullable connection output; null-safe slack message formatting. |
| backend/src/entities/connection/application/dto/created-connection.dto.ts | Make many DTO fields optional/nullable. |
| backend/src/entities/connection/application/data-structures/found-permissions-in-connection.ds.ts | Nullable display_name. |
| backend/src/entities/connection/application/data-structures/found-one-connection.ds.ts | Nullable connectionProperties. |
| backend/src/entities/connection/application/data-structures/found-connections.ds.ts | Make many DS fields optional/nullable. |
| backend/src/entities/connection-properties/utils/validate-create-connection-properties-ds.ts | Guard null default_showing_table. |
| backend/src/entities/connection-properties/utils/build-update-connection-properties-object.ts | Nullable hostname. |
| backend/src/entities/connection-properties/utils/build-connection-properties-entity.ts | Default nullable properties. |
| backend/src/entities/connection-properties/use-cases/update-connection-properties.use.case.ts | Guard missing connection; normalize master password. |
| backend/src/entities/connection-properties/use-cases/find-connection-properties-use.case.ts | Nullable return type. |
| backend/src/entities/connection-properties/use-cases/create-connection-properties.use.case.ts | Guard missing connection; normalize master password. |
| backend/src/entities/connection-properties/use-cases/connection-properties-use.cases.interface.ts | Nullable return type. |
| backend/src/entities/connection-properties/connection-properties.entity.ts | Explicit varchar types for nullable fields. |
| backend/src/entities/connection-properties/application/data-structures/found-connection-properties.ds.ts | Nullable fields + optional connectionId. |
| backend/src/entities/connection-properties/application/data-structures/create-connection-properties.ds.ts | Nullable userId/master_password/table_categories/hostname. |
| backend/src/entities/company-info/utils/build-found-company-info-ds.ts | Null-safe group users; nullable custom domain. |
| backend/src/entities/company-info/use-cases/verify-invite-user-in-company.use.case.ts | Guard missing invited email; guard missing group users. |
| backend/src/entities/company-info/use-cases/update-users-company-roles.use.case.ts | Guard missing users-to-update roles. |
| backend/src/entities/company-info/use-cases/update-company-name.use.case.ts | Guard missing company. |
| backend/src/entities/company-info/use-cases/toggle-test-connections-company-display-mode.use.case.ts | Guard missing company entity on update. |
| backend/src/entities/company-info/use-cases/get-all-users-in-company.use.case.ts | Skip null userInfo DS. |
| backend/src/entities/company-info/use-cases/find-company-white-label-properties.use.case.ts | Nullable subscription level handling. |
| backend/src/entities/company-info/invitation-in-company/invitation-in-company.entity.ts | Explicit varchar types for nullable fields. |
| backend/src/entities/company-info/company-info-helper.service.ts | Null-safe SaaS subscriptionLevel access. |
| backend/src/entities/company-info/application/dto/found-company-white-label-properties.ro.ts | Nullable fields for white-label props. |
| backend/src/entities/company-info/application/dto/found-company-logo.ro.ts | Nullable image info fields. |
| backend/src/entities/company-info/application/data-structures/found-company-info.ds.ts | Nullable logo/favicon/tab_title; optional is2faEnabled. |
| backend/src/entities/cedar-authorization/cedar-permissions.service.ts | Fix map building + typed filter predicates. |
| backend/src/entities/cedar-authorization/cedar-authorization.service.ts | Guard missing IDs; typed filter predicates. |
| backend/src/entities/ai/use-cases/request-info-from-table-with-ai-v7.use.case.ts | Null-safe schema and agent email. |
| backend/src/entities/ai/ai.service.ts | Only set ordering when mapped ordering is not null. |
| backend/src/decorators/slug-verification.decorator.ts | Null-safe verification string validation. |
| backend/src/decorators/slug-uuid.decorator.ts | Null-safe param validation. |
| backend/src/decorators/query-uuid.decorator.ts | Validate query UUID type. |
| backend/src/decorators/query-table-name.decorator.ts | Validate query tableName type. |
| backend/src/decorators/master-password.decorator.ts | Allow null master password. |
| backend/src/decorators/gclid-decorator.ts | Allow null gclid header. |
| backend/src/common/application/global-database-context.ts | Make startTransaction properly awaitable. |
| backend/src/common/abstract-use.case.ts | Ensure _dbContext is initialized before use. |
| backend/src/authorization/utils/extract-token-from-header.ts | Narrow input type to headers-only. |
| backend/src/authorization/temporary-auth.middleware.ts | Nullable token + guard missing secret. |
| backend/src/authorization/saas-auth.middleware.ts | Guard missing microservice JWT secret. |
| backend/src/authorization/non-scoped-auth.middleware.ts | Nullable token + guard missing secret. |
| backend/src/authorization/auth.middleware.ts | Nullable token + guard missing secret. |
| backend/src/authorization/auth-with-api.middleware.ts | Use IRequestWithCognitoInfo; guard missing secret. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| password, | ||
| database, | ||
| host: host ?? undefined, | ||
| port: parseInt(port ?? '', 10), |
Comment on lines
+29
to
+31
| if (!secretKey) { | ||
| throw new BadRequestException('Turnstile secret key is not configured.'); | ||
| } |
Comment on lines
+243
to
+248
| throw new HttpException( | ||
| { | ||
| message: Messages.FAILED_ADD_ROW_IN_TABLE, | ||
| }, | ||
| HttpStatus.BAD_REQUEST, | ||
| ); |
| title: group.title, | ||
| isMain: group.isMain, | ||
| users: group.users?.map((user) => buildSimpleUserInfoDs(user)), | ||
| users: group.users?.map((user) => buildSimpleUserInfoDs(user)).filter((user) => !!user), |
Comment on lines
+21
to
+24
| if (!cognitoUserName) { | ||
| reject(new BadRequestException(Messages.CONNECTION_NOT_FOUND)); | ||
| return; | ||
| } |
Comment on lines
+24
to
+27
| const userId: string | undefined = request.decoded.sub; | ||
| if (!userId) { | ||
| reject(new BadRequestException(Messages.COMPANY_ID_MISSING)); | ||
| return; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.