Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
11 changes: 11 additions & 0 deletions .console/log.md
Original file line number Diff line number Diff line change
Expand Up @@ -88,3 +88,14 @@ truth; pre-push catches regressions before they hit GitHub.
## 2026-05-23 — Standardize pre-push hook

- Updated `.hooks/pre-push` to the auto-discovering boundary-artifact variant (auto-locates PrivateManifest/dist artifact; extra custodian-multi fallback path).

## 2026-06-18 — cleanup: delete dead git-ops primitives + all_patches

Ecosystem incomplete-integration remediation (Phase 3 DELETE). Removed
over-ported dead code from the initial fork-manager port (PR #2): git/git_ops.py
`remote_url` + `list_files_changed_between`, and patches.py
`PatchRegistry.all_patches`. All three: zero src callers, zero tests, not in any
__all__, no cross-repo consumers (verified). Sibling accessors (`for_source`,
`by_source`, `get`) and the wired git primitives stay. 66 tests green; ruff +
audit (B2 env only). (Pre-existing stale T7 doctor warnings for
validation/vocabulary dirs are unrelated and not a CI gate here.)
14 changes: 0 additions & 14 deletions src/source_registry/git/git_ops.py
Original file line number Diff line number Diff line change
Expand Up @@ -79,17 +79,3 @@ def force_push_with_lease(repo: str | Path, remote: str, branch: str) -> GitResu
def is_clean(repo: str | Path) -> bool:
res = _git(repo, "status", "--porcelain")
return res.ok and res.stdout.strip() == ""


def remote_url(repo: str | Path, remote: str) -> str | None:
res = _git(repo, "remote", "get-url", remote)
return res.stdout.strip() if res.ok else None


def list_files_changed_between(
repo: str | Path, base_ref: str, head_ref: str,
) -> list[str]:
res = _git(repo, "diff", "--name-only", f"{base_ref}..{head_ref}")
if not res.ok:
return []
return [line for line in res.stdout.splitlines() if line.strip()]
6 changes: 0 additions & 6 deletions src/source_registry/patches.py
Original file line number Diff line number Diff line change
Expand Up @@ -49,12 +49,6 @@ def by_source(self) -> dict[str, list[PatchRecord]]:
def for_source(self, source_name: str) -> list[PatchRecord]:
return list(self._by_source.get(source_name, []))

def all_patches(self) -> list[PatchRecord]:
out: list[PatchRecord] = []
for patches in self._by_source.values():
out.extend(patches)
return out

def get(self, full_id: str) -> PatchRecord | None:
"""Lookup by ``<source>:<PATCH-NNN>`` identifier."""
if ":" not in full_id:
Expand Down
Loading