Skip to content

[codex] Preserve repos added after setup - #24

Draft
rushillshah wants to merge 1 commit into
mainfrom
agent/connect-repos-preserve-existing
Draft

[codex] Preserve repos added after setup#24
rushillshah wants to merge 1 commit into
mainfrom
agent/connect-repos-preserve-existing

Conversation

@rushillshah

Copy link
Copy Markdown
Owner

Scope

  • Preserves repositories already stored in digital-brain.config.json when connect-repos adds explicit paths later.
  • Deduplicates repeated paths while retaining insertion order.
  • Extends the post-init regression test to cover repeated and additive repository selection.

Partially addresses #3.

Validation

  • npm run check
  • node --test --test-name-pattern='connect-repos can add repository context after init' tests/cli-init.test.mjs
  • npm pack --dry-run
  • npm test (65 passed, 4 pre-existing date-sensitive failures reproduced on origin/main; tracked in Tests use stale dated fixtures and fail as time advances #23)

Risk

Low. The merge applies only when explicit repository paths are supplied. Interactive selection behavior is unchanged.

@rushillshah rushillshah added the bug Something isn't working label Jul 10, 2026
@rushillshah

Copy link
Copy Markdown
Owner Author

Maintenance blocker (2026-07-10): both CI jobs fail on the same four date-sensitive fixture tests reproduced on unchanged origin/main. The focused connect-repos regression passes. Next action: fix #23, then re-run CI; keep this PR draft until checks pass and review is complete.

@rushillshah

Copy link
Copy Markdown
Owner Author

Maintenance update (2026-07-13): the shared date-fixture blocker is fixed in draft PR #25, with npm run check and all 70 tests passing locally. Next action: review/merge #25, then rerun this PR’s CI; keep #24 draft until both checks pass and review is complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant