Skip to content

perf(artifact): run checks only after a push and drop replaced checks - #696

Merged
harlan-zw merged 1 commit into
mainfrom
perf/sweep-on-commit-change
Nov 12, 2026
Merged

harlan-zw merged 1 commit into
mainfrom
perf/sweep-on-commit-change

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

Related to #688

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

#688 slowed the skilld run sweep to a weekly or daily check. A check of an unchanged repository still writes a Resolution with about 60 KB of attestations and proves nothing new.

  • /api/skills/runnable now returns each Skill's revision, which is its repository's pushed_at. The sweep records the revision it checked in a new artifact_run_checks.checked_revision column (migration 0160, additive). A Skill is due again when that revision changes. Backstops cover a change in the checks themselves: 30 days for a ready Skill, 7 days for a failing one. Unchecked Skills and transient failures stay due at once.
  • When a check settles, the sweep deletes the Resolution of the check it replaced, unless that Resolution holds the newest attestation of its Artifact. The predicate is the one the batch prune uses, so the sweep keeps about one Resolution per Skill instead of three days of them.

🤖 AI disclosure: Harlan Agent Kit modified this description. My AI open-source policy.

🤖 Generated with Claude Code

A run check of an unchanged repository proves nothing new, but it still
writes a Resolution with about 60 KB of attestations. Check a Skill again
when its repository's pushed_at changes, with a 30-day backstop for ready
Skills and 7 days for failing ones. When a check settles, delete the
Resolution of the check it replaced unless it holds the newest
attestation of its Artifact, so the sweep keeps one row per Skill.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Nov 12, 2026
@harlan-github-agent

harlan-github-agent Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

🤖 MERGED

Harlan Agent Kit posted this automated review. It is not Harlan's personal review or approval. AI open source policy. Last updated: 2026-11-12 05:51 UTC.

GitHub merged this pull request.

  • Logged (70/100): A transient failure can consume a corrective push and postpone recovery detection for a week. View code Optional follow-up: Optional follow-up: track whether the latest attempt was transient independently of the preserved failure state, and keep that attempt immediately due.
    • Ask an agent to verify and fix this finding
  • Logged (73/100): A metadata refresh can mark a new source revision checked before that source was actually checked. View code Optional follow-up: Optional follow-up: include the admitted commit and folder in the scheduling identity, and record the identity actually requested.
    • Ask an agent to verify and fix this finding

1791b226-fd79-410c-afe2-093adc9365a0

Selected findings run after merge and open separate pull requests.

@harlan-github-agent harlan-github-agent Bot added harlan-agent-ready The automated Review passed every gate on this head commit. and removed harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Nov 12, 2026

@harlan-github-agent harlan-github-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Harlan Agent Kit posted this automated review. AI open source policy.

Automated Review: READY.

Read the Review results

@harlan-zw
harlan-zw merged commit 1795a33 into main Nov 12, 2026
8 checks passed
@harlan-zw
harlan-zw deleted the perf/sweep-on-commit-change branch November 12, 2026 05:44
@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-ready The automated Review passed every gate on this head commit. label Nov 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant