Skip to content

Fix workflow checkpoint restore scoping#150

Open
3em0 wants to merge 1 commit into
AIDC-AI:mainfrom
3em0:fix/checkpoint-session-scope
Open

Fix workflow checkpoint restore scoping#150
3em0 wants to merge 1 commit into
AIDC-AI:mainfrom
3em0:fix/checkpoint-session-scope

Conversation

@3em0
Copy link
Copy Markdown

@3em0 3em0 commented May 27, 2026

Summary

  • require restore/update checkpoint requests to include the current session id
  • scope checkpoint DAO reads and UI updates by both version id and session id
  • scope workflow rewrite checkpoint reads to the active session
  • rebuild checked-in frontend assets so restore/update calls send X-Session-ID

Fixes #149

Verification

  • npm run build
  • python -m compileall backend/dao/workflow_table.py backend/controller/conversation_api.py backend/service/workflow_rewrite_tools.py
  • local end-to-end regression harness: Alice restore succeeds, Bob restore/update of Alice checkpoint fails
  • git diff --check

Note

  • npm run lint is currently blocked because the existing ESLint config references @typescript-eslint/eslint-plugin, which is not installed by npm ci.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 27, 2026

CLA assistant check
All committers have signed the CLA.

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.

Cross-Tenant Data Exposure via Unscoped Workflow Checkpoint Restore

2 participants