Skip to content

Deprecate unused commands/update CLAUDE.md with skill instructions#790

Merged
forstmeier merged 6 commits intomasterfrom
claude-interaction-tweaks
Mar 18, 2026
Merged

Deprecate unused commands/update CLAUDE.md with skill instructions#790
forstmeier merged 6 commits intomasterfrom
claude-interaction-tweaks

Conversation

@forstmeier
Copy link
Copy Markdown
Collaborator

@forstmeier forstmeier commented Mar 10, 2026

Overview

Changes

  • remove unused Claude commands
  • include skill/plugin-specific instructions

Context

I think it's nice to make things more flexible for users and I'm generally leaning fully into conversational flow versus structured systems.

Summary by CodeRabbit

  • Documentation

    • Updated guidance to favor direct GraphQL usage for token-efficient scripts/tools, recommend suggesting commands (e.g., /update-pull-request) when pull-request bots provide feedback, and encourage invoking skills/commands proactively from conversation.
  • Chores

    • Removed legacy interactive issue-creation and update-notes guides and consolidated related documentation processes.

Copilot AI review requested due to automatic review settings March 10, 2026 19:55
@github-project-automation github-project-automation Bot moved this to In Progress in Overview Mar 10, 2026
@github-actions github-actions Bot requested a review from chrisaddy March 10, 2026 19:55
@github-actions github-actions Bot added the markdown Markdown code updates label Mar 10, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 10, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 45d0f36e-2fa9-4d87-91a9-9c33b4bc52ed

📥 Commits

Reviewing files that changed from the base of the PR and between 91ab6c8 and 9dac400.

📒 Files selected for processing (1)
  • CLAUDE.md

📝 Walkthrough

Walkthrough

Removed two .claude command docs and updated CLAUDE.md to favor direct GraphQL usage in scripts/tools, add a recommendation to suggest /update-pull-request when PR review bots provide feedback (with PR number), and encourage proactive invocation/suggestion of skills/commands.

Changes

Cohort / File(s) Summary
Deleted Command Files
.claude/commands/create-or-update-issue.md, .claude/commands/update-notes.md
Deleted two command/automation documentation files that described interactive workflows for creating/updating GitHub issues and consolidating lesson notes.
Guidance Updates
CLAUDE.md
Edited guidance: recommend using GitHub GraphQL directly for scripts/tools, add suggestion to run /update-pull-request when review bots report feedback (including PR reference), and encourage proactive invocation/suggestion of skills/commands.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately captures the main changes: deprecating unused commands (two files deleted) and updating CLAUDE.md with new skill instructions, matching the PR's stated objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude-interaction-tweaks
📝 Coding Plan
  • Generate coding plan for human review comments

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 and usage tips.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Mar 10, 2026

Greptile Summary

This PR removes two unused slash commands (.claude/commands/create-or-update-issue.md and .claude/commands/update-notes.md) and updates CLAUDE.md to reflect a shift toward conversational, context-driven skill invocation rather than explicit command-based workflows.

  • The two deleted command files are no longer referenced anywhere and their removal is clean — the update-pull-request command (.claude/commands/update-pull-request.md) that is newly referenced in CLAUDE.md still exists.
  • The new CLAUDE.md notes are internally consistent: the guidance to suggest /update-pull-request on bot feedback aligns with the remaining command file, and the instruction to invoke skills proactively from conversational context gives a clear directive for the updated workflow philosophy.
  • The lessons consolidation process previously handled by update-notes.md is no longer backed by a dedicated command, but remains achievable via conversational flow per the new guidelines — this trade-off is intentional per the PR description.

Confidence Score: 5/5

  • This PR is safe to merge — all changes are limited to documentation and workflow configuration files with no impact on application code.
  • All three changed files are documentation/configuration only. The deletions are intentional and clearly motivated, the remaining command file referenced by the new CLAUDE.md note exists, and no code logic is affected.
  • No files require special attention.

Important Files Changed

Filename Overview
.claude/commands/create-or-update-issue.md Deleted — structured command for creating/updating GitHub issues removed in favor of conversational workflow.
.claude/commands/update-notes.md Deleted — lessons consolidation command removed; the underlying workflow is still supported conversationally per updated CLAUDE.md guidance.
CLAUDE.md Updated with three new notes: refined GraphQL API guidance, a directive to suggest /update-pull-request on bot feedback, and a new instruction to invoke skills proactively from conversational context.

Last reviewed commit: "Merge branch 'master..."

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s Claude/agent usage guidance by removing unused command definitions and expanding CLAUDE.md with more context-driven instructions for interacting with GitHub and invoking skills/commands.

Changes:

  • Update CLAUDE.md guidance around GitHub queries (MCP vs direct GraphQL) and conversational command suggestion.
  • Remove deprecated .claude/commands/update-notes.md.
  • Remove deprecated .claude/commands/create-or-update-issue.md.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
CLAUDE.md Updates agent guidance on GitHub querying approach and when to suggest /update-pull-request / skill invocation.
.claude/commands/update-notes.md Removes an unused command definition for consolidating lessons into CLAUDE.md.
.claude/commands/create-or-update-issue.md Removes an unused command definition for creating/updating GitHub issues via gh/GraphQL.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CLAUDE.md Outdated
Comment thread CLAUDE.md Outdated
coderabbitai[bot]
coderabbitai Bot previously approved these changes Mar 10, 2026
coderabbitai[bot]
coderabbitai Bot previously approved these changes Mar 11, 2026
Copilot AI review requested due to automatic review settings March 11, 2026 15:25
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings March 14, 2026 13:43
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@forstmeier forstmeier merged commit eab3c93 into master Mar 18, 2026
8 checks passed
@forstmeier forstmeier deleted the claude-interaction-tweaks branch March 18, 2026 22:37
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Overview Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

markdown Markdown code updates

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants