Skip to content

feat(ui): Issues view + sync rendering — every state from the GitHub sync on Issues, Inbox, Work, counts and Molecule view (#266) - #277

Merged
DanielNetzer merged 2 commits into
mainfrom
forgecell/issue-266-d90fd6b27f2de38bbcb7ede3
Oct 27, 2026
Merged

DanielNetzer merged 2 commits into
mainfrom
forgecell/issue-266-d90fd6b27f2de38bbcb7ede3

Conversation

@DanielNetzer

Copy link
Copy Markdown
Owner

Implements issue #266.

Molecule: mol-266-c01981a8ea041f3a5eb0afce
Formula: forgecell-github-issue
Base: 5a638555bff15af112ab3fa92ab4bb8ceacaee8e
Delivery: 7e8804d3ed55804ac245a7a885b6717bcef3d682

Awaiting exact-commit CI and human review. Harness-reported checks are not independent CI evidence. No merge, deployment or issue closure is authorized.

Daniel Netzer and others added 2 commits October 27, 2026 21:24
…sync on Issues, Inbox, Work, counts and Molecule view (#266)
…ion-card gate posts to core-gates.js to keep molecule-decision.js under 600 lines

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@DanielNetzer

Copy link
Copy Markdown
Owner Author

Adversarial review (2026-10-27): no HIGH. Fixed in this slice: Start intake enforces the full startable rule server-side (rows 1/5/10/15) and requires a snapshot fetched after the request began (cancelled request ⇒ not-checked, never the old snapshot); every count and pill derives from the server DisplayState, with a server-side needsYou flag; polling never drops focus in the Issues table; Refresh uses aria-disabled; every row shows an action or a plain reason. JS behavioural tests are tracked in #271.

Merged main (1f51ed5) after #273/#274/#276: kept main's issueSync (it already had the now clock, plus the loop view's last-answer cache) and removed the duplicate field; unioned molecule-decision.js imports; moved the decision card's five gate posts (approve scope, preview, publish, stop, retire) unchanged into core-gates.js so molecule-decision.js stays under the 600-line module limit. Verified on the merged tree: build, vet, go test -count=1 ./internal/ui/... ./internal/forgesync/... (305 passed), node --check on every module.

@DanielNetzer
DanielNetzer marked this pull request as ready for review October 27, 2026 19:25
@DanielNetzer
DanielNetzer merged commit 3146aed into main Oct 27, 2026
2 checks passed
@DanielNetzer
DanielNetzer deleted the forgecell/issue-266-d90fd6b27f2de38bbcb7ede3 branch October 27, 2026 19:35
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.

1 participant