Skip to content

fix(sessions): retain the cause of cold Git sync failures - #4841

Open
ScriptedAlchemy wants to merge 14 commits into
masterfrom
codex/cold-git-sync-diagnostics
Open

ScriptedAlchemy wants to merge 14 commits into
masterfrom
codex/cold-git-sync-diagnostics

Conversation

@ScriptedAlchemy

@ScriptedAlchemy ScriptedAlchemy commented Nov 9, 2026 •

Copy link
Copy Markdown
Owner

This preserves the independent session diagnostic change from Doctor PR #4799. Cold Git sync records the failing operation and its cause through the existing diagnostic path when scope resolution, scope storage or the topology publisher fails. The real CLI fixture includes daemon output on failure and retains its mounted-project and JSON assertions.

Commit 872c5f8a31 carries the original two files from 621940d19e. Its base is master be2e1349c6. The hosted Linux suite for the original combined commit passed 183 targets. The native macOS Bazel cold-daemon CLI journey executed and passed in 10.17s. The full exact-head Linux run is pending. Keep this prerequisite draft for independent review. Original Linux run

Reject empty project selectors and budget the entire finding, including both commands, deletion warning and measured overage. Omit actions that cannot fit intact. Add filesystem-backed malformed/long-selector regressions and sparse 15 GiB reporting coverage with a canonical identity and long Unicode root.

Preserve concurrent formatting commit d7c2d3a. Native Bazel validation remains required; no Cargo commands or CI changes.

Refs #4616
Repair three hosted automation_runner_test failures by supplying the
configured executable identity required by the production prefilter.
Use the same identity for the prefilter and retained runner, preserving
manual-schedule, exact-ledger, live-lock and lock-race assertions.

Add a production-prefilter regression for unconfigured refusal,
deduplication and same-occurrence recovery after binding, with zero
backend calls and no effectful terminal. Do not relax backend admission,
change timeouts, or alter the graph publication changes.

Refs #4532
@changeset-bot

changeset-bot Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 2c0c240

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ScriptedAlchemy
ScriptedAlchemy marked this pull request as ready for review November 9, 2026 23:55
devin-ai-integration[bot]

This comment was marked as resolved.

This branch has not been deployed

No deployments
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.

1 participant