merge main into release - #4732
Conversation
…e + layout split (#4609)
…e tool contracts (#4605)
…(Phase 1 + 3a + 5 groundwork) (#4614)
…rs (Phase 3, slices 1–2) (#4622)
…n the drain loop (#4623)
… reducers (Phase 3, slices 3–4) (#4626)
…odel (usage middleware, G2, build_turn_models) (#4625)
…orchestrator posture) (#4630)
…porting Completed (B6) (#4659)
…5.1 validation (B14) (#4664)
…working receive loop (#4660)
…nput_context dry-run + boolean typing (B13, B7, B8) (#4665)
…kflows (frontend paths) (#4668)
…can't resurrect its webview (#4697)
…schema is absent (B16) (#4702)
…dow + connections sessions (#4713)
…e the local core RPC (#4715)
…he copilot can't re-save the flow (#4628) Co-authored-by: Steven Enamakel <enamakel@tinyhumans.ai>
…hed folder items (#4700) Co-authored-by: Steven Enamakel <enamakel@tinyhumans.ai>
…e_count (#4712) Co-authored-by: Steven Enamakel <enamakel@tinyhumans.ai>
…sing (#4699) Co-authored-by: Steven Enamakel <enamakel@tinyhumans.ai>
Co-authored-by: Steven Enamakel <enamakel@tinyhumans.ai>
#4722) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…executors, wire allowlist (#4725)
|
You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool. What Enabling Code Scanning Means:
For more information about GitHub Code Scanning, check out the documentation. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7659c75dc5
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| function storageKey(flowId: string | null): string { | ||
| const userId = getActiveUserId(); | ||
| const scope = userId ? `${userId}:` : ''; | ||
| return `${scope}${STORAGE_PREFIX}${copilotThreadKey(flowId)}`; |
There was a problem hiding this comment.
Avoid persisting the shared draft copilot key
When flowId is null, this persists the thread id under the fixed copilot-thread:draft key. The draft canvas always passes flowId: null, and saving a draft navigates to the new persisted flow without clearing or migrating this key, so the next unsaved draft for the same user—especially after an app reload now that this uses localStorage—will reopen the previous draft's durable workflow_builder thread and seed the copilot with the wrong workflow history. Persist only saved flow IDs, clear the draft key on save/exit, or include a per-draft identifier.
Useful? React with 👍 / 👎.
Co-authored-by: Steven Enamakel <31011319+senamakel@users.noreply.github.com> Co-authored-by: Cyrus Gray <144336577+graycyrus@users.noreply.github.com> Co-authored-by: oxoxDev <164490987+oxoxDev@users.noreply.github.com> Co-authored-by: CodeGhost21 <164498022+CodeGhost21@users.noreply.github.com> Co-authored-by: sanil-23 <sanil@tinyhumans.ai> Co-authored-by: YellowSnnowmann <167776381+YellowSnnowmann@users.noreply.github.com> Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Co-authored-by: M3gA-Mind <elvin@mahadao.com> Co-authored-by: Steven Enamakel <enamakel@tinyhumans.ai> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Summary
Problem
Solution
Submission Checklist
diff-cover) meet the gate enforced by.github/workflows/ci-lite.yml. Runpnpm test:coverageandpnpm test:rustlocally; PRs below 80% on changed lines will not merge.docs/TEST-COVERAGE-MATRIX.mdreflect this change (orN/A: behaviour-only change)## Relateddocs/RELEASE-MANUAL-SMOKE.md)Closes #NNNin the## RelatedsectionImpact
Related
AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
Commit & Branch
Validation Run
pnpm --filter openhuman-app format:checkpnpm typecheckValidation Blocked
command:error:impact:Behavior Changes
Parity Contract
Duplicate / Superseded PR Handling