Repository navigation
Fix resumable, project-scoped Codex session imports - #7047
Draft
codex-corp wants to merge 2 commits into
Draft
codex-corp wants to merge 2 commits into
codex-corp wants to merge 2 commits into
Conversation
Contributor
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Comment |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Problem
Long Codex conversations can exhaust a bounded ingestion pass before completing a session. File-level cursors alone lose successful window progress, and zero processed sessions can conceal useful work or a provider failure. Project-scoped imports also need trustworthy metadata and identical discovery/import boundaries.
Solution
The driver selects transcripts by metadata before reading the batch. Optional project scope accepts only canonical Codex project directories within the configured root; unknown paths and symlink escapes are excluded. Project state is isolated and an OS lock rejects overlapping imports.
Scoped passes retain five-session/five-call budgets. Successful windows and recovery splits are persisted against content fingerprints; unchanged windows are reused after interruption, while changed windows are reviewed again. A failed session does not block its healthy neighbor and never receives a completed cursor. Persona reduction uses the existing deterministic fallback when the scoped call budget is exhausted.
The host checks processing permission and summarizer availability before dispatch. Optional report fields carry opaque session IDs, stable reason codes and fixed safe remediation text. The UI continues draining checkpoint progress, labels estimated counts, and retains actionable errors after refreshing status.
Dependency source diffs
This single OpenHuman PR is a draft pending maintainer adoption of the published library commits and a TinyMemory module release/artifact-pin update. This PR does not hand-edit release versions or checksums. The installed application and its current import have not been replaced or restarted.
Submission Checklist
Impact
Desktop coding-session ingestion. New serde-default report fields remain compatible with old responses; the TinyMemory bus minor contract advances from 4.1 to 4.2. No filesystem path is added to RPC requests. General imports retain Claude behavior and legacy budgets. No user provider, model, consent or embedding configuration is changed.
Related
AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
Commit & Branch
fix/codex-session-import.a66bee76b6185640f67bef0e4b5fd94935b63a7e.Validation Run
pnpm --filter openhuman-app format:check.pnpm typecheck.cargo check, core Clippy, TinyMemory fmt/Clippy/build/full tests and TinyCortex persona full tests passed.pnpm docs:checkand lint passed; lint reports 68 pre-existing warnings, zero errors.Validation Blocked
command:N/A for the completed checks above.error:None outstanding for those checks.impact:merged diff coverage remains unmeasured locally; dependency module publication remains a release prerequisite.Behavior Changes
Parity Contract
Duplicate / Superseded PR Handling