Skip to content

fix(skills): remove stale upstream lock entries - #178

Merged
shunkakinoki merged 1 commit into
mainfrom
codex/fix-upgrade-skill-sync
Aug 2, 2026
Merged

fix(skills): remove stale upstream lock entries#178
shunkakinoki merged 1 commit into
mainfrom
codex/fix-upgrade-skill-sync

Conversation

@shunkakinoki

@shunkakinoki shunkakinoki commented Aug 2, 2026

Copy link
Copy Markdown
Owner

The skills lock contained five upstream entries that cannot be installed:

  • analytics-tracking was renamed to analytics
  • Graphify has no valid SKILL.md
  • next-browser has invalid skill frontmatter
  • next-skills has no valid skills

Update SKILLS.txt and skills-lock.json so skills-install only processes valid current sources.


Summary by cubic

Cleaned up SKILLS.txt and skills-lock.json to remove invalid upstreams and rename analytics-tracking to analytics. This makes skills-install only process valid sources and prevents install errors.

  • Bug Fixes
    • Renamed coreyhaines31/marketingskills analytics-tracking to analytics.
    • Removed Graphify-Labs/graphify (no valid SKILL.md).
    • Removed vercel-labs/next-browser (invalid skill frontmatter).
    • Removed vercel-labs/next-skills entries (no valid skills).

Written for commit 8a6c236. Summary will update on new commits.

Review in cubic

@coderabbitai

coderabbitai Bot commented Aug 2, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 023cd7f3-c398-49bf-8d58-23960d3000d3

📥 Commits

Reviewing files that changed from the base of the PR and between 0ef0e70 and 8a6c236.

📒 Files selected for processing (2)
  • SKILLS.txt
  • skills-lock.json

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated the marketing skills selection to use analytics instead of analytics-tracking.
    • Removed outdated Graphify Labs and Vercel Labs skill repository entries.
    • Synchronized the skills configuration and lock data.

Walkthrough

The selected marketing skill changed from analytics-tracking to analytics. The Graphify-Labs and Vercel Labs repository entries were removed from SKILLS.txt. Matching entries were updated in skills-lock.json.

Changes

Skill declaration synchronization

Layer / File(s) Summary
Update selected repositories and skills
SKILLS.txt
The marketing selection now uses analytics. The Graphify-Labs and Vercel Labs repository entries were removed.
Synchronize locked skill metadata
skills-lock.json
The lockfile now records analytics with its updated path and hash. The graphify and next-* entries were removed.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Poem

A rabbit checks the skill list bright,
analytics hops into sight.
Graphify leaves without a trace,
Next skills clear their garden space.
The lockfile follows, neat and right.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the removal of stale upstream lock entries, which is the main change.
Description check ✅ Passed The description accurately explains the renamed and removed skill entries and the reason for each change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/fix-upgrade-skill-sync

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cubic-dev-ai cubic-dev-ai 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.

No issues found across 2 files

Re-trigger cubic

@shunkakinoki
shunkakinoki merged commit bfc6007 into main Aug 2, 2026
4 checks passed
@shunkakinoki
shunkakinoki deleted the codex/fix-upgrade-skill-sync branch August 2, 2026 15:07
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