Add card estimates and derived participant rollups - #3054
Conversation
…93-estimate-portability
…93-estimate-editor
…93-estimate-portability
…93-estimate-portability
…93-estimate-portability
…93-estimate-portability
…93-estimate-portability
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 51e12925c3
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
All reviewed source commits are preserved in pushed integration Durable evidence remains under |
|
Hosted required run 34707276360 passed backend, Windows API, frontend, migration/security and other required jobs, then failed two Paper E2E tests at their initial fixed 40-Tab opener setup. Local actual-API Chromium reproduced both failures and recorded the active-element sequence: the new Estimates control pushed the first card beyond that bound. This is an evidenced setup regression, not a flaky classification. Head 7aba61d changes only the two setup paths and STATUS. Each now focuses the opener, steps back to its preceding control, then enters with an actual Tab; the exact focus-visible outline and J/Enter selected-card assertions remain unchanged. Final narrow run: 2 passed, 17.9 seconds. ESLint for the spec, 700-file relative doc links, docs governance and diff whitespace pass. No unrelated runtime or gate changes. A fresh required hosted run is owed at this head. Baseline/final logs and traces were copied and hash-verified under C:/td0912-evidence/estimate-focus-gate (18 local files). They contain synthetic session material and remain local, outside Git/public attachments. Playwright-owned servers exited. The archive child #3059 has merged this parent correction without conflict edits and must still follow oldest-base-first delivery. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 958e9947a0
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 66a9f3e83d
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…93-estimate-noop-pin
|
Card-create reconciliation now preserves a board-detail commit that arrives before the POST continuation, instead of appending the same ID twice or increasing its column count again. It also avoids inserting a late result into a different selected board. The first fix exposed an existing fresh-store integration contract. Final Hosted Windows and Linux at the previous head each failed only that same fresh-store test (6,857 passed, one failed, three skips). Their logs are retained locally. Final real-API Paper/Legacy estimate journeys and the new hosted head gate remain pending. Refs #3062. |
|
Merged as 54e4c0a after required workflow34713059470 passed at reviewed head28c4f49de3c761b1212a4313d8746488e28918f0. The merge preserves #3053/#3055; GitHub confirms issues2093/3044/3049/3062 closed and the seven delivery items are Done. First post-merge check: zero unresolved threads on #3054/#3053/#3055. The archive and relation children have been retargeted to main; their changed bases must pass fresh hosted qualification. No release or physical-device/screen-reader acceptance is inferred. Bounded followups3056/3057/3058/3063/3065 and OUTSTANDING_TASKS.md remain open. |
|
Post-merge worktree closeout: the finished estimate integration tree was clean, its tip28c4f49de is preserved by verified remote main54e4c0a86, and all18 retained focus-gate logs/traces/SQLite evidence files were copied and SHA-256 checked under C:/td0912-evidence/delivered-estimate-integration. The copied-hashes.json and pre-removal.json receipts identify every survivor. Plain git worktree remove completed; the local source branch remains. Other worktrees with earlier automatic cleanup denials were excluded and remain preserved. |
Cards now carry an optional effort estimate, and board readers can inspect current board, column, participant and unassigned totals. Blank remains unknown, explicit zero stays known, and hours/minutes inputs are available in shared card details and Paper/Legacy quick-create. Multiple assignees each receive the full estimate with a visible overlap explanation; parent estimates remain independent.
The complete vertical covers direct writes, reviewed proposals, chat/MCP, audit/provenance, board/account portability and additive SQLite migration/rollback. Intentional estimate changes retain caller version pins and explicit set/clear semantics. Same-valued proposals also perform an atomic persisted-version check before accepting a no-op, preserving unchanged versions and avoiding false successful receipts after competing writes. Ordered same-card operations and planned creates remain supported.
The editor preserves newer drafts while advancing successful-save versions. Quick-create reconciles with a detail payload that arrives before its POST continuation, preventing duplicate cards/counts while preserving fresh-store creation and navigation ownership. The integration also preserves reviewed source PRs #3053 (planning/backup references) and #3055 (permission recovery).
Validation:
Exact final-head required hosted CI must pass before merge. Separate bounded follow-ups retain backend-less demo totals (#3056), rollup snapshot consistency (#3058), request timeout/recovery (#3063), and older-reader export protection (#3065). Physical-device/screen-reader acceptance, skipped PostgreSQL/Docker cases and release/hosting decisions remain in OUTSTANDING_TASKS.md.
Closes #2093
Closes #3044
Closes #3049
Closes #3062