Skip to content

Shop room: refresh the vendored widget from Chronicle - #161

Merged
keyxmakerx merged 2 commits into
mainfrom
claude/project-thread-5tj6tg
Oct 4, 2026
Merged

keyxmakerx merged 2 commits into
mainfrom
claude/project-thread-5tj6tg

Conversation

@keyxmakerx

@keyxmakerx keyxmakerx commented Oct 4, 2026 •

Copy link
Copy Markdown
Owner

Fixes: none (keeps vendor/chronicle/ identical to Chronicle after keyxmakerx/Chronicle#1057)
Security implication: none. This is a client-side widget copy only. Buying and the socket relay are untouched.
Consumer-verified: n/a
Mockup: n/a. Nothing changes on screen in Foundry: the new panel exists only in arrange mode, and the module never turns arrange mode on.

What this changes

Before: the module's copy of Chronicle's shop room widget was older than Chronicle main.

After: vendor/chronicle/shop_room.js is byte-identical to Chronicle main, which now includes #1057's slide-out arrange panel.

In Foundry the room looks and works the same as before. The module never sets data-can-arrange, so the Arrange button and panel don't render. The new outside-click listener returns straight away unless arrange mode is on, and the widget's own cleanup removes it.

How

Test plan

  • CHRONICLE_DIR=../Chronicle node --test tools/test-*.mjs with Chronicle at current main (2896bd1): 1417 pass, 0 fail. The run includes tools/test-shop-room.mjs, which checks that the copy is identical to Chronicle's.
  • CI is green on 5a0c1e9 (bench, check-descriptor, check-error-catalog).
  • Live Foundry check: TESTING.md "Shop Widget" (no new steps).

Tenet self-check

  • T-B1 security: no new path.
  • T-B2 plugin isolation: vendored file only.
  • T-B3 production UI: Chronicle's own widget, unchanged.
  • T-B4 docs: no behavior change in Foundry, so no docs to change.

claude added 2 commits October 4, 2026 00:20
Chronicle's shop room widget gained a slide-out arrange panel that waits
for Save room. Foundry never turns on arrange mode, so nothing changes on
screen here; the copy is refreshed so it stays identical to Chronicle.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JRAx4Ex5fEHkxVB5Yaqw86
@keyxmakerx
keyxmakerx marked this pull request as ready for review October 4, 2026 04:26
@keyxmakerx
keyxmakerx merged commit a3e973b into main Oct 4, 2026
3 checks passed
@keyxmakerx
keyxmakerx deleted the claude/project-thread-5tj6tg branch October 4, 2026 05:01
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.

2 participants