Skip to content

chore: complete bounded manual RepoWise analysis - #645

Merged
efraespada merged 6 commits into
developfrom
chore/644-optimize-manual-repowise-analysis-and-complete-bounded-dead-code-and-history-scans
Oct 21, 2026
Merged

efraespada merged 6 commits into
developfrom
chore/644-optimize-manual-repowise-analysis-and-complete-bounded-dead-code-and-history-scans

Conversation

@efraespada

@efraespada efraespada commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Manual RepoWise previously timed out dead-code and exhausted memory while loading historical blobs. Reuse one completed index for health/dead-code and scan history through a bounded two-worker queue, with complete coverage evidence and a failing status for interrupted analysis.

What changed

  • Pin RepoWise 0.58.0 and export stored health Markdown/JSON and safe-only dead-code results at the analyzed SHA. Preserve 500 commits for index git analytics independently of unlimited reachable security history.
  • Keep full reachable-history secret analysis, including removed secrets and generated bundles. Read one blob per worker, retain at most two tasks/results, consume results in upstream order, and preserve upstream detectors, allowlists, provenance and deduplication.
  • Apply explicit phase time/virtual-memory budgets; reject stale indexes, malformed output, failed Git/persistence operations, incomplete coverage and timeouts. Publish scope, timings, resource evidence and completed reports with unconditional cleanup/upload.
  • Keep manual-only admission and read-only permissions. Update workflow contracts, SDD/catalog evidence and operator documentation.

Validation

Native manual run 39583408904 passed on 6a6dc1227ab1dea54b930b996fe8b7e25a13c3d2. All five phases are complete; the summary and artifact 12339775726 were inspected.

Phase Seconds Largest-process peak RSS (MiB)
Index 300.527 1144.5
Health Markdown 1.371 110.4
Health JSON 1.474 110.5
Dead-code 1.421 100.9
History security 1028.631 320.4
  • Coverage: 4,244 reachable commits, 22,994 source blobs, 9,512,087,107 bytes; two workers, zero reader blob cache, largest blob 7,999,775 bytes. Published commit/read counts match the independent completeness checks.
  • The job completed in 23m15s with complete scans. Baseline run 39573977241 took 20m37s with incomplete dead-code and security; its index alone took 8m06s. Completion adds real historical scanning rather than hiding partial results.
  • 39 Python compatibility/resource/failure regressions pass locally and on native Ubuntu; 271 workflow-contract tests pass. Full platform coverage/acceptance checks pass on Ubuntu, macOS and Windows. Workflow, documentation, agent-docs, docs-assets and specification validators pass; typecheck/lint pass; Graphify updated.
  • Action-owned synchronization from develop passed in run 39581065235. Final required CI Check 39583416977: passed on the same revision.
  • Artifact SHA256: d1297167c412c661873221f8efc7625149e76a544da2aa0d077f46f8a4a441e1 (14-day retention).

Review notes

The adapter uses explicitly pinned private RepoWise interfaces; upgrades must pass differential compatibility fixtures. Security remains advisory and follows RepoWise's supported source-extension/pattern/allowlist scope. Its budget is 1200 seconds with a 2 GiB coordinator cap and 1 GiB per worker, inside the 40-minute job. RSS above is the largest process peak, not aggregate simultaneous memory.

Closes #644

@docs-page

docs-page Bot commented Oct 21, 2026

Copy link
Copy Markdown

To preview the documentation for this pull request, visit the following URL:

docs.page/vypdev/copilot~645

Documentation is deployed and generated using docs.page

@vypbot vypbot added size: XL Indicates that the issue or task is extra large in scope and complexity. 75% Progress: 75% labels Oct 21, 2026
@vypbot

vypbot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Copilot approval · Human review needed

Status: A protected workflow, policy, or trust-boundary file changed.
Completed: 1/1 observed checks succeeded; Bugbot complete.
Action required: Ask a human reviewer to decide.
Impact: No new approval is confirmed for this revision.
See checks

Technical evidence

Revision: 6a6dc1227ab1; base: b6fba19fe9b9; reason: protected-path.

@vypbot

vypbot commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

Bugbot: review complete

Current status: No active findings on 6a6dc12.

Pull request · Verified commit · Workflow run

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.10%. Comparing base (274db40) to head (6a6dc12).
⚠️ Report is 1564 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #645      +/-   ##
===========================================
+ Coverage    90.94%   94.10%   +3.16%     
===========================================
  Files          135      950     +815     
  Lines         6251    30548   +24297     
  Branches      1338     8965    +7627     
===========================================
+ Hits          5685    28748   +23063     
- Misses         172      704     +532     
- Partials       394     1096     +702     

see 1017 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vypbot vypbot added size: XXL Indicates an extremely large task or issue that requires significant time and effort to complete. 80% Progress: 80% and removed size: XL Indicates that the issue or task is extra large in scope and complexity. 75% Progress: 75% labels Oct 21, 2026
@efraespada
efraespada merged commit 78861c2 into develop Oct 21, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

80% Progress: 80% size: XXL Indicates an extremely large task or issue that requires significant time and effort to complete. state:verified

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants