Skip to content

java-compile: keep the engine scan across runs (Part of #5142) - #6605

Merged
squid-shepherd[bot] merged 1 commit into
mainfrom
churn/5142-20261025T0657Z
Oct 25, 2026
Merged

squid-shepherd[bot] merged 1 commit into
mainfrom
churn/5142-20261025T0657Z

Conversation

@squid-protocol

@squid-protocol squid-protocol commented Oct 25, 2026 •

Copy link
Copy Markdown
Owner

Wires --scan-cache (added in #6375) into the Java workflows so the dsf engine scan (~90 s per side) is restored instead of re-run.

  • java-base-generation.yml: new java-scan-<corpus>-<hash> cache (lookup-only check, --scan-cache on the seeding run, main-only save). Generation is re-run when either the base-generation entry or the scan entry is missing.
  • java-compile.yml: key step + restore-only (ci: java-same-base and gradle-home caches save near-duplicate entries (sibling of #4847) #4968), --scan-cache on the "unchanged?" check and the engine-scan build. Fallback restore-keys prefix per corpus; the tool's scan_key is the exact check, so a coarse Actions key can only cost a rescan, never a wrong scan.
  • Key = scan-deciding code (translator and the two controllers left out, the det/copy stand-ins and cics_task_containers.py kept in) + corpus pin + matrix tool. PR hashes the base checkout, push its own.
  • tests/tools/test_cache_keys.py: pins restore/save split, equal hash inputs on both sides, flags passed.

Cross-repo: none.

Measurement (ci-workflows rule 5): no new job; adds one lookup-only cache restore per seeding run. Expected saving on a warm hit: up to ~90 s per side on dsf in the unchanged check. Not measured on CI yet (first effect after the seeding workflow runs on main). Plan reach unchanged (same path filters).

Checklist: partial [c:7a2199c1] -- the seed fires only on translator/controller-path pushes, so after an engine-only main commit a PR still misses and rescans; the dsf ticket pass (~20 s/config) remains.

Gates: pr_gates --fast 7/7, audit_check clear, workflow tests 391 pass.

Lessons for agents

  • hashFiles joined with - inside one ${{ }} is subtraction: one ${{ }} per part.

🤖 Generated with Claude Code

Class / Sibling gaps

Copied from the worker's report by promoter-bot's precheck.

Sibling gaps

  • Seeding the scan on engine-only main pushes, which would add six jobs per push. No burned-estate program needs it; it is CI speed only (infra scope).
  • The dsf ticket pass.

…che (Part of #5142)

java-base-generation.yml seeds a content-keyed scan cache per corpus, java-compile.yml restores it
(restore-only) and passes --scan-cache to the unchanged check and the engine-scan build.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@squid-protocol squid-protocol added shepherd:merge Maintainer: merge when green (auto-merge, squash). Guide: docs/ci.md and removed shepherd:merge Maintainer: merge when green (auto-merge, squash). Guide: docs/ci.md labels Oct 25, 2026
@squid-shepherd

Copy link
Copy Markdown
Contributor

No description provided.

@squid-shepherd
squid-shepherd Bot marked this pull request as ready for review October 25, 2026 07:06
@github-actions

Copy link
Copy Markdown
Contributor

🐦‍⬛ Muninn Security Scan

✅ No security issues found.

🐦‍⬛ Powered by Muninn · Skald Lab

@squid-shepherd

Copy link
Copy Markdown
Contributor

No description provided.

@squid-shepherd
squid-shepherd Bot enabled auto-merge (squash) October 25, 2026 07:17
@squid-shepherd

Copy link
Copy Markdown
Contributor

No description provided.

@squid-shepherd
squid-shepherd Bot merged commit 918b2a4 into main Oct 25, 2026
75 checks passed
@squid-shepherd
squid-shepherd Bot deleted the churn/5142-20261025T0657Z branch October 25, 2026 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

shepherd:merge Maintainer: merge when green (auto-merge, squash). Guide: docs/ci.md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant