Skip to content

chore: exclude stale build artifacts from rexec sync; reuse the projection plan; align a pin comment #226

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
chrisdpurcell merged 2 commits into from
Sep 1, 2026
Merged

chore: exclude stale build artifacts from rexec sync; reuse the projection plan; align a pin comment #226

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1822828
chore: exclude stale build artifacts from rexec sync; align the markd…
chrisdpurcell Sep 1, 2026
597a5c4
perf(projection): reuse the computed plan in sync-payload-projection …
chrisdpurcell Sep 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion .rexec.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,12 @@ git_context = true
# Leg worktrees and orchestration state are workstation-only: mirroring them
# made rexec abort with "local source tree changed continuously" whenever a
# dispatched worker was editing in .claude/worktrees/ (2026-08-26).
exclude = [".claude/worktrees", ".workflow"]
#
# build/dist, build/release-dist, build/release-sdist are stale release
# artifacts (289 MiB) that rexec would otherwise mirror on every sync;
# build/wheel-runtime and build/release-wheel stay included because
# scripts/verify.sh:133-141 needs exactly one candidate wheel there.
exclude = [".claude/worktrees", ".workflow", "build/dist", "build/release-dist", "build/release-sdist"]

[tools]

Expand Down
11 changes: 8 additions & 3 deletions src/project_standards/package_contract/projection.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -183,9 +183,11 @@ def _finding(code: str, plan: ProjectionPlan, path: Path, message: str) -> Packa
)


def projection_findings(root: Path) -> tuple[PackageFinding, ...]:
def projection_findings(
root: Path, *, plan: ProjectionPlan | None = None
) -> tuple[PackageFinding, ...]:
"""Return projection drift findings without modifying any filesystem entry."""
plan = plan_payload_projection(root)
plan = plan if plan is not None else plan_payload_projection(root)
expected = {link.destination: link for link in plan.links}
canonical_roots = {
plan.root / f"standards/{link.standard_id}/versions/{link.version}"
Expand Down Expand Up @@ -330,7 +332,10 @@ def sync_payload_projection(
) -> tuple[PackageFinding, ...]:
"""Check read-only, or reconcile only projection symlinks and empty directories."""
plan = plan_payload_projection(root)
findings = projection_findings(plan.root)
# Reuse the plan just computed instead of re-scanning manifests and
# schemas a second time inside projection_findings (measured ~3.5s
# redundant over 212 manifests + 416 schemas).
findings = projection_findings(plan.root, plan=plan)
if check or not findings:
return findings
blocking = {
Expand Down
2 changes: 1 addition & 1 deletion tests/coherence/test_pins.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

_REPO = Path(__file__).resolve().parent.parent.parent
PRETTIER_PIN = "3.9.6"
MARKDOWNLINT_CLI2_PIN = "0.23.2" # Keep aligned with markdownlint-cli2-action@v24.
MARKDOWNLINT_CLI2_PIN = "0.23.2" # Keep aligned with markdownlint-cli2-action@v24.2.0.


def test_package_json_pins_both_tools() -> None:
Expand Down
Loading