Skip to content

Refine feature identity and state actions - #10

Merged
tiye merged 1 commit into
mainfrom
codex/feature-identity
Aug 30, 2026
Merged

Refine feature identity and state actions#10
tiye merged 1 commit into
mainfrom
codex/feature-identity

Conversation

@tiye

@tiye tiye commented Aug 30, 2026

Copy link
Copy Markdown
Member

Summary

  • install opaque ambient feature identity at feature_root(...), with feature_key() and feature_marker(name) for descendants
  • remove panel_key forwarding from Todo/Lab helpers and keep multi-instance store, listener, effect, and DOM marker isolation
  • make Todo save and Lab reply domain actions carry their current component values as versioned serializable payloads
  • keep component-store transitions local to the originating event instead of letting domain reducers/workflows read or write child stores
  • remove the unused public cross-component store read/write/clear API and feature-specific path coordination helpers
  • update README, developer guidance, plan, tests, and GitHub tracking

Verification

  • yarn test:koka — 64/64 pass
  • yarn check — Koka compile and Vite production bundle pass
  • browser smoke: Todo edit/save with escaped payload characters
  • browser smoke: Lab reply sends the typed message and clears its component draft
  • browser smoke: component draft survives Koka HMR regeneration and a full reload through localStorage
  • browser console: no errors

Closes #7.

Follow-ups are tracked separately in #8 (action replay) and #9 (framework lifecycle cleanup for unreachable keyed child stores).

Copilot AI lite review requested due to automatic review settings August 30, 2026 05:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@tiye
tiye merged commit 8cba523 into main Aug 30, 2026
1 check passed
@tiye
tiye deleted the codex/feature-identity branch August 30, 2026 05:53
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.

隐藏 feature identity 并移除跨组件 store path 协调 / Hide feature identity and remove cross-component store path coordination

2 participants