Skip to content

fix: retry failed connector updates and preserve Git path history - #553

Merged
mavaali merged 2 commits into
mainfrom
codex/fix-retry-and-git-paths
Sep 6, 2026
Merged

fix: retry failed connector updates and preserve Git path history#553
mavaali merged 2 commits into
mainfrom
codex/fix-retry-and-git-paths

Conversation

@mavaali

@mavaali mavaali commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Failed connector updates could persist a new revision with the old completed hash, causing subsequent syncs to skip the failed work. Changed revisions now retain the existing empty-hash pending marker until distillation succeeds. Tests cover returned errors, thrown errors, persisted checkpoint recovery, and skipping the revision after successful recovery.

Bulk Git history previously parsed quoted, newline-delimited paths and trimmed trailing whitespace. It now uses NUL-delimited fields and preserves exact pathnames. Regression coverage includes accented/CJK names, tabs, embedded and leading newlines, trailing spaces, control-byte prefixes, multiple commits and an interleaved empty commit.

Closes #534.
Closes #538.

Validation: build passed; full suite 4,827 passed / 17 skipped; focused suites passed; lint passed with six pre-existing informational findings and no changed-file findings. Existing installed dependencies were reused; this is not a clean-install dependency validation. Both regressions failed before the fixes.

This prevents future stuck checkpoints; previously persisted falsely-complete connector revisions are not migrated automatically and require a targeted refresh. Coordinate the small engine change with the shared integration work in #511/#514.

@mavaali
mavaali merged commit bb66675 into main Sep 6, 2026
13 checks passed
@mavaali mavaali mentioned this pull request Sep 6, 2026
mavaali added a commit that referenced this pull request Sep 6, 2026
Minor release. Adds declared frontmatter filtering in vault_search (#513),
fixes connector-update retry + exact Git path-history preservation (#553),
and folds in dependency/CI bumps since v3.12.1. Also realigns manifest.json,
which drifted at 3.12.0 when the 3.12.1 security patch skipped it.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

fix(attest): preserve Git history for Unicode and special-character filenames fix(integrations): retry failed source revision updates

1 participant