rbrooks/Quest-Board

Watch

Fork 0

D30 is settled, not provisional (#659) #687

Closed

claude-bot wants to merge 0 commits from docs/settle-d30 into main

claude-bot commented

2026-09-17 23:50:05 +00:00

Collaborator

Refs #659. Small, but worth not leaving.

D30 — the three note panels becoming one with an audience switch — was recorded as provisional because you asked to see it drawn before settling it. It was drawn in lane 3 and reviewed on 17 September with no changes asked for.

Left as "provisional" it reaches the rebuild as an open question that had already been closed, and someone would have had to come back and ask you again.

Updates the intro's decisions table, the SESS-7 header, and the lane 3 page's own copy, which still said "you asked to see it before settling it".

Catalogue rebuilt: 1,586 rows on 102 surfaces (unchanged — wording only).

🤖 Generated with Claude Code

Refs #659. Small, but worth not leaving. **D30** — the three note panels becoming one with an audience switch — was recorded as *provisional* because you asked to see it drawn before settling it. It was drawn in lane 3 and reviewed on 17 September with no changes asked for. Left as "provisional" it reaches the rebuild as an open question that had already been closed, and someone would have had to come back and ask you again. Updates the intro's decisions table, the `SESS-7` header, and the lane 3 page's own copy, which still said "you asked to see it before settling it". Catalogue rebuilt: 1,586 rows on 102 surfaces (unchanged — wording only). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

claude-bot added 1 commit

2026-09-17 23:50:05 +00:00

docs(catalogue): D30 is settled, not provisional (#659 )

CI / Bot/backend version sync (pull_request) Successful in 26s

Details

CI / Feedback site tests (pull_request) Successful in 48s

Details

CI / Backend lint (ruff) (pull_request) Successful in 1m23s

Details

CI / Docker image build (pull_request) Successful in 2m57s

Details

CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 3m2s

Details

CI / Bot tests and audit (pull_request) Successful in 4m47s

Details

CI / Frontend tests, audit, and build (pull_request) Failing after 6m44s

Details

CI / Synthetic session harness (no GPU, no LLM) (pull_request) Successful in 12m32s

Details

CI / Backend migration, tests, and audit (pull_request) Successful in 19m39s

Details

ade12d7ed5

The notes panel -- one editor with an audience switch, replacing the three
separate panels -- was recorded provisional because the owner asked to see it
drawn before settling it. It was drawn in lane 3 and reviewed on 17 September
with no changes asked for.

Left as "provisional" it would reach the rebuild as an open question that had
already been closed.

Refs #659.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

rbrooks added 1 commit

2026-09-18 01:04:45 +00:00

test(frontend): 30s for the two tests that render 250 entries (#684 )

CI / Bot/backend version sync (pull_request) Successful in 26s

Details

CI / Feedback site tests (pull_request) Successful in 41s

Details

CI / Backend lint (ruff) (pull_request) Successful in 1m20s

Details

CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 2m22s

Details

CI / Bot tests and audit (pull_request) Successful in 3m51s

Details

CI / Frontend tests, audit, and build (pull_request) Successful in 4m35s

Details

CI / Docker image build (pull_request) Successful in 4m54s

Details

CI / Backend migration, tests, and audit (pull_request) Successful in 12m38s

Details

CI / Synthetic session harness (no GPU, no LLM) (pull_request) Successful in 13m0s

Details

1ed505a8d2

CI failed on a docs-only PR -- seven insertions and seven deletions across
three catalogue files and one prototype page -- with two frontend tests
timing out:

  FAIL CampaignStoryline.test.jsx > shows every article past the backend's
       200-entry page   Error: Test timed out in 15000ms.
  FAIL WikiArticle.test.jsx       > shows every article past the backend's
       200-entry page   Error: Test timed out in 15000ms.
  Test Files  2 failed | 65 passed (67)

Neither can be reached by that diff. Both render 250 entries and drive a
real userEvent click, and both pass locally in 1.7s and 1.3s -- so about 6s
on a runner measured at roughly 4x slower than a dev machine (#684). That
fits inside 15s right up until another suite runs beside it, which is what
happened: the pytest v9 PR's run overlapped this one.

Raised to 30s with a comment saying why, matching the override
WikiDraftReview.test.jsx already uses for three of its own. A re-run would
have gone green and taught nobody anything; these two are the only tests in
the suite that build a page that size, and they were one contended run away
from doing this again.

Verified: 37 passed across both files.

Refs #684.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

rbrooks referenced this pull request from a commit

2026-09-18 01:24:04 +00:00

Merge PR #687: D30 is settled, and 30s for two tests that render 250 entries (#659, #684)

claude-bot commented

2026-09-18 01:24:08 +00:00

Author

Collaborator

Merged to main in cff380b. Closing by hand; the API merge returns 422 on this repo.

The first run failed on two frontend tests timing out at 15s — on a diff of seven insertions across three catalogue files and one prototype page, which cannot reach either test. Both render 250 entries and drive a real userEvent click; they take 1.7s and 1.3s locally, so about 6s on this runner, which fits inside 15s until another suite runs beside them. #688's run overlapped this one.

Raised to 30s with the reason written down, matching the override WikiDraftReview.test.jsx already uses. They are the only two tests in the suite that build a page that size.

Merged to `main` in cff380b. Closing by hand; the API merge returns 422 on this repo. The first run failed on two frontend tests timing out at 15s — on a diff of seven insertions across three catalogue files and one prototype page, which cannot reach either test. Both render 250 entries and drive a real `userEvent` click; they take 1.7s and 1.3s locally, so about 6s on this runner, which fits inside 15s until another suite runs beside them. #688's run overlapped this one. Raised to 30s with the reason written down, matching the override `WikiDraftReview.test.jsx` already uses. They are the only two tests in the suite that build a page that size.

claude-bot closed this pull request

2026-09-18 01:24:12 +00:00

claude-bot referenced this pull request

2026-09-18 02:11:10 +00:00

D33 — scenes become first-class and Foundry-pairable (#659, #689) #690