Skip to content

Restore Room message actions on the direct page - #80

Merged
NWYLZW merged 1 commit into
codex/chatroom-74-ui-ownershipfrom
codex/chatroom-74-message-actions
Sep 8, 2026
Merged

NWYLZW merged 1 commit into
codex/chatroom-74-ui-ownershipfrom
codex/chatroom-74-message-actions

Conversation

@NWYLZW

@NWYLZW NWYLZW commented Sep 8, 2026

Copy link
Copy Markdown
Member

Change

The direct Room page previously ignored ordinary message actions, while the retained Host Shell rendered the first two actions inline and placed the remainder in an overflow menu. This branch restores that behavior on the plugin-owned page without reintroducing a Shell source.

The UI shows two direct actions, preserves action order, displays remaining actions in the existing keyboard-accessible menu, exposes disabled reasons, fences duplicate clicks while a command is pending, reports failures and permits a later retry. Execution re-resolves the action by Room, message and action ID from the current projection before calling the owner-bound public ctx.commands.execute service. Removed, disabled, stale or disposed actions fail before command dispatch; invocation keys are encoded tuples so special characters cannot alias another action.

Current Chatroom production projections emit empty ordinary-message action arrays, so the behavioral tests inject protocol-valid actions to exercise this compatibility path. The change does not add a fabricated product action or a second persistence ledger.

This is an isolated review delta on top of the #73/#74 ownership candidate in PR #78. Native product acceptance and old Host Shell retirement remain separate gates.

Validation

  • npm run check — 373/373 tests pass, including CSS lint, typecheck and production build
  • npm run format:check
  • npm run lint:source
  • focused page source, timeline behavior and wiring tests — 21/21 pass

@NWYLZW
NWYLZW force-pushed the codex/chatroom-74-message-actions branch from 37fc6bc to 06150e3 Compare September 8, 2026 13:10
@NWYLZW
NWYLZW merged commit 06150e3 into codex/chatroom-74-ui-ownership Sep 8, 2026
6 checks passed
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