Skip to content

feat(registry): retain Skill description history - #594

Merged
harlan-zw merged 1 commit into
mainfrom
research/skill-description-history
Oct 9, 2026
Merged

harlan-zw merged 1 commit into
mainfrom
research/skill-description-history

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

❓ Type of change

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

📚 Description

Source refresh overwrites author descriptions, preventing comparisons across versions. Preserve complete frontmatter by content hash, with source identity and observation times, including missing descriptions and invocation settings.

Backfill current complete snapshots and capture later snapshots through database triggers. Repository moves carry the history to the new identity; source removal retains it.

Source refresh writes the current snapshot. Database triggers retain description history, which follows repository moves.

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

@harlan-zw harlan-zw added the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 9, 2026
@harlan-github-agent harlan-github-agent Bot added harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. harlan-agent-running An Agent holds a Task on this issue or pull request right now. and removed harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. labels Oct 9, 2026
@harlan-github-agent

harlan-github-agent Bot commented Oct 9, 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-10-09 03:46 UTC.

GitHub merged this pull request.

  • Logged (8/100): The screenshot documenting the history-capture and repository-move data flow is unavailable, so the description's visual claim is unverified. View code
    • Ask an agent to verify and fix this finding
  • Logged (12/100): Observation times are write times, not presence times: unchanged content never extends last_observed_at even though syncs confirm the snapshot remains current. View code
    • Ask an agent to verify and fix this finding

414346ba-f013-4abf-8149-a38733801baf

Selected findings run after merge and open separate pull requests.

@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-running An Agent holds a Task on this issue or pull request right now. label Oct 9, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

Written by the Agent.

Replayed migration 0150 against the frozen public-source export in SQLite.
The export contained 20,510 Skill rows. The migration retained 20,151 complete versions with zero duplicate keys.
The replay took 43 milliseconds locally. This does not measure production D1 migration time.

Behavior checks cover changed descriptions, repeated content, missing descriptions, incomplete snapshots, source removal, repository moves, and backfill.

@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-review-required Pull request triage requires an adversarial Review for this head commit. labels Oct 9, 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 ebf6a73 into main Oct 9, 2026
8 checks passed
@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-ready The automated Review passed every gate on this head commit. label Oct 9, 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