Skip to content

[sync] T8028 support explicit null ordering in structured sorts - #3674

Open
tea-artist wants to merge 1 commit into
developfrom
sync/ee-20261021-042027
Open

tea-artist wants to merge 1 commit into
developfrom
sync/ee-20261021-042027

Conversation

@tea-artist

@tea-artist tea-artist commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

🔄 Automated sync from EE repository.

9 commit(s) synced since last sync.

Authors

Included commits

  • T8028 support explicit null ordering in structured sorts (nichenqin)
  • T8023 refresh conditional rollup targets when a computed filter field changes (nichenqin)
  • T8025 show App query impact before deleting a field (nichenqin)
  • T1311 Unify the Command-K palette for project, view, and AI actions (nichenqin)
  • chore(deps): upgrade the MCP SDK and follow-redirects to clear advisories (Chambers)
  • fix(oauth): answer 401 when a refresh token fails verification (nichenqin)
  • T8009 reject conditional filters the target field cannot evaluate (nichenqin)
  • T8007 complete structured query P0: dependency index, coverage report and generation eval (nichenqin)
  • fix(ai): run every send on the model its selector shows, kept per chat T7881 (Pengap)

Latest source commit: teableio/teable-ee@9fa6704

This PR was automatically created by the sync workflow.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-10-21T04:20:32.906994Z
We couldn’t complete this review. Comment @codex review to try again.
16c054e PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@tea-artist tea-artist changed the title [sync] fix(ai): run every send on the model its selector shows, kept per chat T7881 [sync] T8007 complete structured query P0: dependency index, coverage report and generation eval Oct 21, 2026
@tea-artist
tea-artist force-pushed the sync/ee-20261021-042027 branch from 16c054e to 8ee99d5 Compare October 21, 2026 04:31
@coveralls

coveralls commented Oct 21, 2026 •

Copy link
Copy Markdown

Coverage Report for CI Build 39628086546

Coverage increased (+0.05%) to 47.044%

Details

  • Coverage increased (+0.05%) from the base build.
  • Patch coverage: 78 uncovered changes across 5 files (176 of 254 lines covered, 69.29%).
  • 17 coverage regressions across 7 files.

Uncovered Changes

File Changed Covered %
apps/nestjs-backend/src/features/base-sql-executor/structured-coverage.spec.ts 36 0 0.0%
apps/nestjs-backend/src/features/base-sql-executor/base-sql-executor.service.spec.ts 21 0 0.0%
apps/nestjs-backend/src/features/base-sql-executor/structured-coverage.ts 170 157 92.35%
apps/nestjs-backend/src/features/ai/ai.service.spec.ts 6 0 0.0%
apps/nestjs-backend/src/features/base-sql-executor/base-sql-executor.service.ts 13 11 84.62%
Total (7 files) 254 176 69.29%

Coverage Regressions

17 previously-covered lines in 7 files lost coverage.

File Lines Losing Coverage Coverage
apps/nestjs-backend/src/features/undo-redo/stack/undo-redo-operation.service.ts 10 40.34%
apps/nestjs-backend/src/features/undo-redo/operations/update-view.operation.ts 2 4.76%
apps/nestjs-backend/src/features/field/open-api/field-open-api.service.ts 1 34.84%
apps/nestjs-backend/src/features/undo-redo/operations/convert-field.operation.ts 1 7.14%
apps/nestjs-backend/src/features/undo-redo/operations/create-fields.operation.ts 1 13.33%
apps/nestjs-backend/src/features/view/open-api/view-open-api.service.ts 1 34.13%
apps/nestjs-backend/src/share-db/share-db.adapter.ts 1 87.98%

Coverage Stats

Coverage Status
Relevant Lines: 89257
Covered Lines: 41990
Line Coverage: 47.04%
Relevant Branches: 14048
Covered Branches: 9829
Branch Coverage: 69.97%
Branches in Coverage %: No
Coverage Strength: 288.5 hits per line

💛 - Coveralls

@tea-artist tea-artist changed the title [sync] T8007 complete structured query P0: dependency index, coverage report and generation eval [sync] T8009 reject conditional filters the target field cannot evaluate Oct 21, 2026
@tea-artist
tea-artist force-pushed the sync/ee-20261021-042027 branch from 8ee99d5 to 6ad09d6 Compare October 21, 2026 05:20
@tea-artist tea-artist changed the title [sync] T8009 reject conditional filters the target field cannot evaluate [sync] fix(oauth): answer 401 when a refresh token fails verification Oct 21, 2026
@tea-artist
tea-artist force-pushed the sync/ee-20261021-042027 branch from 6ad09d6 to 14015d7 Compare October 21, 2026 07:46
@tea-artist tea-artist changed the title [sync] fix(oauth): answer 401 when a refresh token fails verification [sync] chore(deps): upgrade the MCP SDK and follow-redirects to clear advisories Oct 21, 2026
@tea-artist
tea-artist force-pushed the sync/ee-20261021-042027 branch from 14015d7 to 15e16fb Compare October 21, 2026 09:01
@tea-artist tea-artist changed the title [sync] chore(deps): upgrade the MCP SDK and follow-redirects to clear advisories [sync] T1311 Unify the Command-K palette for project, view, and AI actions Oct 21, 2026
@tea-artist
tea-artist force-pushed the sync/ee-20261021-042027 branch from 15e16fb to c793076 Compare October 21, 2026 12:46
@tea-artist tea-artist changed the title [sync] T1311 Unify the Command-K palette for project, view, and AI actions [sync] T8025 show App query impact before deleting a field Oct 21, 2026
@tea-artist
tea-artist force-pushed the sync/ee-20261021-042027 branch from c793076 to 75b0e0b Compare October 21, 2026 13:10
@tea-artist tea-artist changed the title [sync] T8025 show App query impact before deleting a field [sync] T8023 refresh conditional rollup targets when a computed filter field changes Oct 21, 2026
@tea-artist
tea-artist force-pushed the sync/ee-20261021-042027 branch from 75b0e0b to fc11152 Compare October 21, 2026 13:13
Synced from teableio/teable-ee@9fa6704

Co-authored-by: Chambers <chambers@teable.ai>
Co-authored-by: Pengap <penganpingprivte@gmail.com>
Co-authored-by: nichenqin <nichenqin@hotmail.com>
@tea-artist tea-artist changed the title [sync] T8023 refresh conditional rollup targets when a computed filter field changes [sync] T8028 support explicit null ordering in structured sorts Oct 21, 2026
@tea-artist
tea-artist force-pushed the sync/ee-20261021-042027 branch from fc11152 to 1ca0aea Compare October 21, 2026 15:42

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants