Open
Conversation
- Add skill_references table (id, skill_id, title, filename, url, type, content) - Add scripts JSON column and fts_content computed column to skills table - Migration 0008: schema changes for references & scripts - Seed endpoint: upsert references with URL validation, Vectorize indexing - GitHub fetcher script: discover refs/scripts via Trees API, resumable - API: return references metadata + parsed scripts in skill detail - UI: SkillReferencesSection and SkillScriptsSection components - CLI: --include-refs and --include-scripts flags, ANSI escape sanitization - CLI: split display logic into use-display.ts for maintainability - Vectorize: index reference titles + first paragraph only (~200K vectors) - Fix: double .where() in api.skill-detail.ts favorites query
…ripts - Add skill_references table to schema docs - Document scripts JSON + fts_content columns - Add references & scripts data flow to system architecture - Update CLI docs with --include-refs/--include-scripts flags - Update CLAUDE.md with new routes and patterns
- C1: sanitize reference content before DB insert (content-scanner) - C1: change onConflictDoNothing to onConflictDoUpdate for refs re-seed - H1: human mode shows refs/scripts by default (raw mode still opt-in) - H2: DRY — use shared fetchSkillReferences in api.skill-detail.ts - H3: add console.warn on scripts JSON parse failure - H4: pass includeRefs/includeScripts through searchAndUse fallback
Complete reference for all 6 commands (search, use, find, publish, report, config) with options, examples, API endpoints, and error handling.
Add KV-backed checkpoint/resume system for batch recomputing composite scores across all skills. Includes: - recompute-all-skills.ts: batch processing with N+1 avoidance - recompute-state.ts: KV checkpoint management (24h TTL auto-cleanup) - api.admin.recompute.ts: GET /progress, POST /run-batch, POST /run-all - Support for resume, custom offsets, batch sizes, max-iterations guard Prevents Worker timeout via 50-batch limit per request. Admin secret auth via X-Admin-Secret header only.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
skill_referencestable andscripts/fts_contentcolumns to skillsSkillReferencesSectionandSkillScriptsSectioncomponents on skill detail page--include-refsand--include-scriptsflags with ANSI escape sanitizationuse-display.tsfor 200 LOC complianceKey decisions
fts_contentcolumn (content + ref titles), doesn't mutate stored SKILL.md--top-n=50flag for progressive GitHub fetchingTest plan
skillx use author/skill --include-refs --include-scripts