ui(webapp): "table notes", now that the fold holds chatter too (#603) #606

Open

claude-bot wants to merge 1 commit from ui/table-notes-label into main

claude-bot commented

2026-09-11 15:55:06 +00:00

Contributor

Follow-up to #603, deferred there until #602 had merged, since #602 rewrote the same component.

#603 widened table_admin from "the table running the game" to "not part of the story", so off-topic chatter is folded there too; run 8 folded the Phasmophobia talk. The event log's labels no longer fit, so:

The fold line: "N table-admin notes" becomes "N table notes".
The row tag: "table admin" becomes "table".
The CHANGELOG: the unreleased #596 entry is updated to match.

The test also changed shape. The row-tag assertion now finds the tag element instead of a word in the row's text. The row renders its parts without spaces ("00:03:10tableGM — …"), so a word-boundary match can't see it.

Checked: SessionDetail.test.jsx passes (81 of 81), and ESLint is clean on both files.

🤖 Generated with Claude Code

Follow-up to #603, deferred there until #602 had merged, since #602 rewrote the same component. #603 widened `table_admin` from "the table running the game" to "not part of the story", so off-topic chatter is folded there too; run 8 folded the Phasmophobia talk. The event log's labels no longer fit, so: - **The fold line:** "N table-admin notes" becomes "**N table notes**". - **The row tag:** "table admin" becomes "**table**". - **The CHANGELOG:** the unreleased #596 entry is updated to match. **The test also changed shape.** The row-tag assertion now finds the tag element instead of a word in the row's text. The row renders its parts without spaces ("00:03:10tableGM — …"), so a word-boundary match can't see it. **Checked:** `SessionDetail.test.jsx` passes (81 of 81), and ESLint is clean on both files. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

claude-bot added 1 commit

2026-09-11 15:55:06 +00:00

ui(webapp): "table notes", now that the fold holds chatter too (#603 )

CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 48s

Details

CI / Docker image build (pull_request) Successful in 47s

Details

CI / Backend lint (ruff) (pull_request) Successful in 48s

Details

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

Details

CI / Frontend tests, audit, and build (pull_request) Failing after 2m20s

Details

CI / Bot tests and audit (pull_request) Successful in 2m23s

Details

CI / Backend migration, tests, and audit (pull_request) Successful in 8m19s

Details

CI / Synthetic session harness (no GPU, no LLM) (pull_request) Successful in 22m37s

Details

9bd8c7de4f

#603 widened the table_admin kind from "the table running the game" to
"not part of the story", so off-topic chatter is folded there as well. The
event log's one quiet line said "N table-admin notes" and each unfolded row
was tagged "table admin", which now misnames the Phasmophobia talk. They
read "N table notes" and "table"; the unreleased #596 changelog entry says
the same.

The row-tag test now finds the tag element instead of a word in the row's
text: the row renders without spaces between its parts ("00:03:10tableGM"),
so a word-boundary match could never see it.

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

claude-bot scheduled this pull request to auto merge when all checks succeed

2026-09-11 15:55:14 +00:00