Repository navigation
feat: add submission foundation - #1494
Draft
Cedric Vidal (cedricvidal) wants to merge 4 commits into
Draft
Cedric Vidal (cedricvidal) wants to merge 4 commits into
Cedric Vidal (cedricvidal) wants to merge 4 commits into
Conversation
Introduce stable submission identities, deterministic request/run vectors, resumable CAS rollover with immutable history, named-handler dependency evaluation, Cosmos-safe indexes, and attempt-scoped worker claim/completion helpers. Align the observation handler identity with the pp-observations dependency contract. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 99f84f04-d3b0-40f1-a254-4dfbd21e4d87
Finalize every native submit as an exact submission vector, recover interrupted final writes, reconcile and archive vector drift, dispatch only current attempts, and expose project-scoped submission detail, history, and retry APIs with polling repair. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 99f84f04-d3b0-40f1-a254-4dfbd21e4d87
Navigate successful submits to a Submission Detail shell, link grouped runs and run detail back to their parent submission, surface readiness and blocking pairs, and add CLI list/get/history/retry parity with Storybook coverage. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 99f84f04-d3b0-40f1-a254-4dfbd21e4d87
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 99f84f04-d3b0-40f1-a254-4dfbd21e4d87
5 of 6 tasks
Cedric Vidal (cedricvidal)
added this pull request to stack #1508
October 15, 2026 15:06
Cedric Vidal (cedricvidal)
removed this pull request from stack #1508
October 15, 2026 22:45
Cedric Vidal (cedricvidal)
added this pull request to stack #1485
October 15, 2026 22:45
Test Results (Node.js 22)test: Run #272
❌ Some tests failed!
|
Cedric Vidal (cedricvidal)
removed this pull request from stack #1485
October 17, 2026 00:57
Collaborator
Author
This branch has not been deployed
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.
Summary
Adds PR A of the submission/readout stack:
SubmissionDocumentidentity with an exact immutable request/run vector per embedded current attempt and immutablesubmission-runshistory;pp-readoutregistration, queue, claim, supersession, and completion contracts over authoritativepp-observationsstate;scope submissionCLI parity.This draft intentionally stops at the submission foundation. The readout worker/report generation implementation belongs in stacked PR B.
Demos
Before
N/A — submissions had no stable detail surface or CLI commands on the base branch.
After
N/A — no recording was produced for this foundation draft.
Testing
pnpm vitest run packages/shared/src/submissions/submission-state.test.ts packages/shared/src/submissions/submission-coordinator.test.ts packages/shared/src/scripts/register-handler.test.ts packages/db-migrations/src/034-create-submission-indexes.test.ts packages/db-migrations/src/required-migrations.test.ts apps/api/src/routes/submissions.test.ts apps/portal/src/lib/enrichment.test.ts apps/portal/src/components/SubmissionStatusCard.test.tsx apps/workers/post-processor/src/handlers/taxonomy-handler.test.ts apps/scheduler/src/handler-dispatcher.test.ts— 73/73 tests.pnpm --filter shared build && pnpm --filter telemetry build && pnpm --filter api build && pnpm --filter scheduler build && pnpm --filter cli build && pnpm --filter db-migrations build && pnpm --filter post-processor build.pnpm --filter portal build-storybook.pnpm --filter api generate:openapi && pnpm --dir website build.pnpm vitest run apps/judge/src/agent-trajectory.test.ts apps/judge/src/observation-trajectory.integration.test.ts— 39/39 discovered tests.pnpm cli submission --help; confirmedlist,get,runs, andretry.pnpm vitest run apps/api/src/endpoints.test.ts— 168/171 passed. The three failing observation-criterion creation tests return 400 instead of 201 and are unchanged from the stacked base behavior.pnpm --filter portal buildfails atapps/portal/src/pages/CriteriaList.tsx:168becauseCriteriaBadgeis not imported; that file is unchanged in this branch.eslintexecutable (spawn ENOENT).Documentation and compatibility
docs/architecture/app-design.mdanddocs/architecture/overview.md.pp-observationshandler identity.website/src/openapi/scope-openapi.json.034-create-submission-indexes; it uses Cosmos-compatible index helpers and a no-op down migration.pp-taxonomy/taxonomytopp-observations/observations. Deploy the updated registration manifest with the matching post-processor build.Checklist
up()/down()and keep it CosmosDB-compatible.NOTICE/NOTICE-REVIEW.txtwithpnpm noticeas needed. (N/A — no dependency changes.)