Skip to content

Reduce Git subprocesses in the codemap layout watcher test - #499

Open
morluto wants to merge 3 commits into
repoprompt:mainfrom
morluto:test/workspace-codemap-isolation
Open

Reduce Git subprocesses in the codemap layout watcher test#499
morluto wants to merge 3 commits into
repoprompt:mainfrom
morluto:test/workspace-codemap-isolation

Conversation

@morluto

@morluto morluto commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Problem

The codemap layout watcher regression test starts six Git subprocesses while preparing its fixture. Much of that work establishes repository eligibility, but the behavior under test is reclassification after a .git layout change. Repeating unrelated production probes makes an already integration-heavy suite slower.

Approach

Keep the parts that establish the regression contract real: a valid repository, the .git transition, watcher delivery, binding-engine readiness, and the preflight assertion. Control only the unrelated eligibility decision through the existing test seam.

Changes

  • Reduce fixture setup from six Git subprocesses to three.
  • Use command-scoped Git identity and signing configuration.
  • Use the controlled eligibility seam.
  • Preserve the real filesystem and watcher path.
  • Normalize ledger resource tags without changing scenario count or historical timing.

This changes test setup only; production behavior is unaffected.

Validation

Repository guardrails and contribution preflights passed. Focused XCTest execution and ledger verification remain macOS CI checks.

@morluto
morluto marked this pull request as ready for review July 11, 2026 06:28
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

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