Skip to content

fix(skills): drop removed trailofbits ask-questions skill - #181

Merged
shunkakinoki merged 1 commit into
mainfrom
fix/drop-stale-trailofbits-ask-questions
Aug 4, 2026
Merged

fix(skills): drop removed trailofbits ask-questions skill#181
shunkakinoki merged 1 commit into
mainfrom
fix/drop-stale-trailofbits-ask-questions

Conversation

@shunkakinoki

@shunkakinoki shunkakinoki commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Upstream removed ask-questions-if-underspecified; keep analytics-tracking.


Summary by cubic

Removed upstream-deleted ask-questions-if-underspecified from trailofbits/skills and switched coreyhaines31/marketingskills to analytics-tracking to keep the skills list valid.

  • Bug Fixes
    • Updated SKILLS.txt to drop the removed skill and reference analytics-tracking.
    • Synced skills-lock.json: removed the ask-questions entry and updated the analytics path and folder hash.

Written for commit d69bbd0. Summary will update on new commits.

Review in cubic

Upstream removed ask-questions-if-underspecified; keep analytics-tracking.
@shunkakinoki
shunkakinoki enabled auto-merge (squash) August 4, 2026 22:51
@shunkakinoki
shunkakinoki disabled auto-merge August 4, 2026 22:51
@shunkakinoki
shunkakinoki merged commit fafc946 into main Aug 4, 2026
1 of 2 checks passed
@shunkakinoki
shunkakinoki deleted the fix/drop-stale-trailofbits-ask-questions branch August 4, 2026 22:51
@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: dfeb5a1f-0872-4744-8fce-80b2dedc8193

📥 Commits

Reviewing files that changed from the base of the PR and between 62a8823 and d69bbd0.

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

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated the selected analytics capability to a more specific analytics-tracking option.
    • Removed an unnecessary clarification skill from the configured toolkit.
    • Refreshed the associated configuration metadata to keep the available capabilities aligned.

Walkthrough

The PR changes the selected marketing skill from analytics to analytics-tracking, updates its lock entry, and removes ask-questions-if-underspecified from the Trail of Bits selection.

Changes

Skill configuration

Layer / File(s) Summary
Selection and lock update
SKILLS.txt, skills-lock.json
The configuration selects analytics-tracking, removes ask-questions-if-underspecified, and updates the locked skill path and hash.

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

Possibly related PRs

Poem

A rabbit checks the skills at night,
Tracking analytics just right.
One question skill hops away,
The lock file follows without delay.
“All set!” says Bun, and bounds from sight.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/drop-stale-trailofbits-ask-questions

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.

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