Skip to content

chore(skills): resync vendored skill mirror - #152

Merged
vraspar merged 1 commit into
mainfrom
A1igator/skill-resync
Aug 14, 2026
Merged

chore(skills): resync vendored skill mirror#152
vraspar merged 1 commit into
mainfrom
A1igator/skill-resync

Conversation

@A1igator

@A1igator A1igator commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Scope

skills/tenjin/SKILL.md is regenerated from https://tenjin.blog/skills.md, clearing the skill-drift failure that currently reds every open PR touching skills/**. Risk is a copy change nobody read, so the whole diff is quoted below: live dropped "parked candidates" from the frontmatter description and from the CLI-pointer paragraph, three lines in each direction, frontmatter otherwise unchanged.

The content diff:

-description: ... for publishing, for updating, and for questions about sales, drafts, or parked candidates. ...
+description: ... for publishing, for updating, and for questions about sales or drafts. ...

-takes sales, drafts, and parked candidates. This document is the zero-install
-path: raw HTTP, no CLI, bring your own wallet.
+takes sales and drafts. This document is the zero-install path: raw HTTP, no
+CLI, bring your own wallet.

Testing

  • node scripts/sync-skill.mjs then git diff --exit-code -- skills/, exactly what the skill-drift job runs: clean at this commit.
  • pnpm format:check, pnpm lint, pnpm typecheck: pass.
  • pnpm vitest run src/skills-text.test.ts src/lib/skills-source.test.ts src/evals-fixtures.test.ts src/skill-resync-pr.test.ts: 79 passed. These are the tests that read skill text; the rest of the suite cannot see this file.

Definition of Done

Upstream tenjin.blog/skills.md dropped the "parked candidates" wording from
the description and the CLI pointer paragraph. Regenerated with
`pnpm sync:skill`; no hand edits.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Pe6pvZ9qzb26EW2a88e1Sp

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@A1igator A1igator added the priority: high High priority label Aug 14, 2026
@vraspar
vraspar merged commit dc521be into main Aug 14, 2026
3 checks passed
@vraspar
vraspar deleted the A1igator/skill-resync branch August 14, 2026 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: high High priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants