Skip to content

Protect registered ports and make page card numbers unambiguous - #81

Merged
changl merged 3 commits into
changl/fixesfrom
changl/fix-server
Nov 1, 2026
Merged

changl merged 3 commits into
changl/fixesfrom
changl/fix-server

Conversation

@changl

@changl changl commented Nov 1, 2026 •

Copy link
Copy Markdown
Owner

Sandbox servers could claim an unbound port reserved for another page, and different card publishers could reuse a page number. Refuse server startup on another page's registered port and make automatic allocation skip reservations; reserve card numbers under the existing store lock across single creates, batches, and edits, with a conflict receipt naming the occupied comment/version and next free number.

Existing duplicate stores remain unchanged: live/current cards retain their base number and older duplicates use labels such as #41·2, leaving new plain numbers available. CLI, browser badges, resolve/withdraw, and finding lookup agree. Re-publishing the same numbered question at the same anchor updates the same card, and unnumbered re-asks retain approval lineage.

Validation:

  • Both beads reproduced on 064b1d5 (2.28.0) with failing regressions before product edits; real duplicate 10/12/41 history checked through read-only temporary snapshots.
  • Non-browser chunks: A–F 712 passed/2 skipped; G–P 616 passed; Q–Z 370 passed; monitor 16 passed.
  • Full browser chunks ran: A–F 101 passed/2 failed, G–P 45 passed/1 failed, Q–Z 91 passed/2 failed. The approval/numbering failures are corrected and covered by a fresh 10-test approval+UI run plus the final suffix-badge regression; the outside-link timeout passed an isolated rerun.
  • Two remaining browser failures reproduce on isolated, exact 2.28.0 source: cross-tab draft propagation and the dark desktop ticket-popup test. The coordinator owns these separately.
  • Ruff, changed JS syntax, and whitespace checks pass. Unreleased changelog entries added; live pages were never written, restarted, or published.
  • GitHub Checks passed on e5a2e52; ready for review against changl/fixes.

Beads: agent-annotate-zv7, agent-annotate-lw1 (left open as requested).

@changl
changl marked this pull request as ready for review November 1, 2026 06:08
@changl
changl merged commit f7f056f into changl/fixes Nov 1, 2026
1 check passed
@changl changl mentioned this pull request Nov 2, 2026
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