Split process-library into create-library + process-library#101
Merged
Conversation
barefootford
force-pushed
the
split-process-library-create-library
branch
2 times, most recently
from
c8d662f to
749844e
Compare
The process-library skill mixed two concerns: creating/scaffolding a new library (settings, gather project info, Library.create) and processing footage (caffeinate, analyze-video, readiness, backup). Split them: - create-library (new): settings init, gather project info, scaffold the library, then hand off to process-library. - process-library (rewritten): locate/resume an existing library, add footage, run analysis, verify readiness, backup. Also updates AGENTS.md workflow steps and the skills/.gitignore allowlist for the new shipped skill. x sq
barefootford
force-pushed
the
split-process-library-create-library
branch
from
749844e to
fe3f7df
Compare
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.
Splits the
process-libraryskill into two skills along the create/process seam:create-library(new): one-time settings init, gather project info (name, footage location, language, transcript proofreading), andLibrary.create. On completion it hands off toprocess-libraryto analyze the footage, so a new user's experience stays one continuous flow.process-library(rewritten): locate/resume an existing library, optionally add footage, then caffeinate → analyze-video → readiness check → backup → stop caffeinate.process-librarywas doing two genuinely independent jobs — creating/scaffolding a new library and processing footage. They have different entry points (start a new project vs. resume/add footage/analyze) and bundling them made the skill harder to trigger precisely and longer than it needed to be. Splitting them mirrors the pattern the rest of the codebase already follows.Also changed
AGENTS.mdworkflow steps: now Create Library → Process Library → Edit → Backup.skills/.gitignoreallowlist: adds the shippedcreate-libraryskill (both the/and/**entries).