Skip to content

adding prediction gating for resonance questions - #187

Open
mrbdahlem wants to merge 1 commit into
mainfrom
codex-syncdeck-resonance-prediction-gating
Open

adding prediction gating for resonance questions#187
mrbdahlem wants to merge 1 commit into
mainfrom
codex-syncdeck-resonance-prediction-gating

Conversation

@mrbdahlem

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings March 26, 2026 02:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds an implementation plan document for “prediction gating” when embedding Resonance inside SyncDeck, outlining a host/child responsibility split and a proposed cross-iframe capability contract to disable SyncDeck overlay navigation until required Resonance prediction submissions are complete.

Changes:

  • Adds a detailed plan for extending embedded launch options to include prediction gating directives.
  • Proposes a generic child→host postMessage contract (activebits-embedded/navigationCapabilities) for navigation enable/disable signals.
  • Defines an initial Resonance unlock rule (“unlock once all active questions are submitted”) plus a phased implementation/testing checklist.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +60 to +68
```ts
{
type: 'activebits-embedded',
action: 'navigationCapabilities',
payload: {
canGoForward?: boolean,
canGoDown?: boolean
}
}

Copilot AI Mar 26, 2026

Copy link

Choose a reason for hiding this comment

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

The proposed child→host navigationCapabilities message payload only includes canGoForward/canGoDown, but SyncDeck already models 4-direction navigation capabilities (canGoBack|canGoForward|canGoUp|canGoDown). To keep the contract generic and consistent with existing capability naming, consider including all four optional fields (or explicitly document why only forward/down are supported).

Copilot uses AI. Check for mistakes.
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