feat(yaml): extract Docker Compose services and GitHub Actions jobs - #2541
feat(yaml): extract Docker Compose services and GitHub Actions jobs#2541bruno-growthsales wants to merge 1 commit into
Graphify — looks good
Looks safe to merge — no coupling regressions and no blocking issues, checked against the code graph (not a self-assessment).
Details
Graphify reviewed this change.
Looks safe to merge — no coupling regressions and no blocking issues, checked against the code graph (not a self-assessment).
Graphify review — findings
This PR expands CI/CD and repository tooling for the graphify project. It adds several GitHub Actions workflows (a skillgen artifact-consistency check, PyPI trusted publishing, a self-graph release asset builder, and security scanning via bandit/pip-audit), switches the existing test workflow from pip to uv with full-history checkouts, and broadens the trigger branches. It also introduces supporting config files: .dockerignore, .gitattributes (Linguist HTML handling), .github/FUNDING.yml, a pre-commit hook running the skillgen check, and expanded .gitignore entries. The changed-symbols list additionally references many test, extractor, and skillgen-related files across the codebase, suggesting accompanying source/test/fixture changes not shown in the truncated diff. Reviewers should look at the full diff for those areas since only the CI/config portion is visible here.
No blocking issues surfaced.
Analysis details — impact, health, verification
Impact & health
Graphify review
Impact — 12643 functions depend on the 12624 functions this change touches.
Health — this change adds coupling hotspots:
- worse:
extract()— 373 callers, 39 callees - worse:
_rebuild_code()— 93 callers, 52 callees - worse:
build_from_json()— 153 callers, 17 callees - worse:
detect()— 82 callers, 13 callees - new:
deduplicate_entities()— 49 callers, 21 callees - new:
build_merge()— 41 callers, 14 callees - worse:
save_semantic_cache()— 50 callers, 9 callees - new:
_extract_generic()— 18 callers, 20 callees - …and 198 more
Verification — 12643 functions in the blast radius were not formally verified this run (proofs are advisory here).
Gate & verification
graphify gate
PASS — objectively clean (no health regressions, tests not run — proofs not run this pass (advisory)). Grounded, not self-assessed.
Advisory (not blocking):
- verification_scope: 12643 function(s) in the blast radius were not formally verified this run