Skip to content

propagate project assignments to new dependencies - #19551

Draft
andz-bb wants to merge 21 commits into
project-dependencies-backendfrom
codex/project-dependency-propagation
Draft

andz-bb wants to merge 21 commits into
project-dependencies-backendfrom
codex/project-dependency-propagation

Conversation

@andz-bb

@andz-bb andz-bb commented Aug 25, 2026 •

Copy link
Copy Markdown
Contributor

Description

keeps project assignments current as resources change, on top of #19245.

  • adds project memberships to newly referenced dependencies across resource save paths
  • preserves exclusions during unchanged saves, duplication, history restores and repaired screen saves
  • includes existing dependencies when a resource gains another project membership
  • updates agent query tools and preserves exclusions when queries move between datasources
  • validates REST template imports before changing existing connections
  • serialises assignment-sensitive writes, keeping external schema discovery and row transfers outside the assignment lock

covered by propagation, query-move, REST import and screen-save tests.

part 3 of the six-part projects stack. review against the preceding branch. next: #19246.

Addresses

App Export

  • n/a

Screenshots

  • n/a

Launchcontrol

keeps project dependencies assigned as resource relationships change

@andz-bb andz-bb linked an issue Aug 25, 2026 that may be closed by this pull request
@andz-bb
andz-bb force-pushed the codex/project-dependency-propagation branch from 4c77b24 to df54d90 Compare August 27, 2026 10:40

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 15 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread packages/server/src/api/controllers/table/index.ts Outdated
Comment thread packages/builder/src/stores/builder/automations.ts
Comment thread packages/server/src/api/controllers/query/index.ts Outdated
Comment thread packages/server/src/api/controllers/screen.ts
@andz-bb

andz-bb commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai rerun

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai rerun

@andz-bb I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 18 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread packages/server/src/api/controllers/query/index.ts
Comment thread packages/server/src/api/controllers/query/index.ts Outdated

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 18 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@andz-bb
andz-bb force-pushed the codex/project-dependency-propagation branch from c40087c to 4d9073e Compare September 2, 2026 08:59
@andz-bb

andz-bb commented Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai rerun

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai rerun

@andz-bb I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 25 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread packages/server/src/api/controllers/query/index.ts
Comment thread packages/server/src/api/controllers/datasource.ts
Comment thread packages/server/src/api/routes/tests/project.spec.ts Outdated
Comment thread packages/server/src/sdk/workspace/projects/dependencies.ts
@andz-bb

andz-bb commented Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai rerun

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai rerun

@andz-bb I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 28 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread packages/server/src/api/routes/tests/project.spec.ts
@andz-bb

andz-bb commented Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai rerun

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai rerun

@andz-bb I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 28 files

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Fix all with cubic | Re-trigger cubic

Comment thread packages/server/src/api/routes/tests/project.spec.ts Outdated
@andz-bb

andz-bb commented Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai rerun

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai rerun

@andz-bb I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 28 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Re-trigger cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 26 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread packages/server/src/api/routes/tests/project.spec.ts Outdated
@andz-bb
andz-bb force-pushed the codex/project-dependency-propagation branch from 1c38109 to c880420 Compare September 23, 2026 12:19
@github-actions github-actions Bot removed the stale label Sep 23, 2026
@andz-bb
andz-bb force-pushed the codex/project-dependency-propagation branch from da77e4b to 260dfe1 Compare September 28, 2026 15:00
@andz-bb
andz-bb force-pushed the codex/project-dependency-propagation branch from 260dfe1 to c133210 Compare September 29, 2026 19:11
@andz-bb
andz-bb force-pushed the codex/project-dependency-propagation branch 2 times, most recently from 53e120f to c18815b Compare October 1, 2026 10:26
@andz-bb
andz-bb force-pushed the codex/project-dependency-propagation branch from c18815b to 4db6b3b Compare October 1, 2026 10:36

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Projects

1 participant