Skip to content

Remove the @claude workflow - #146

Open
simpleanalytics-agent wants to merge 1 commit into
mainfrom
feature/remove-claude-workflow
Open

simpleanalytics-agent wants to merge 1 commit into
mainfrom
feature/remove-claude-workflow

Conversation

@simpleanalytics-agent

@simpleanalytics-agent simpleanalytics-agent commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Fixes #145

Removes .github/workflows/claude.yml, so Claude Code no longer runs on @claude mentions. CodeRabbit now reviews pull requests. Merge this before github-actions#11, which deletes the shared claude.yml this file calls.

Security implications

  • No security impact
  • Has security impact - described as: removes an automation that gave Claude Code write access to contents, pull requests, and issues when someone mentioned @claude. This reduces automation permissions.

Testing

  • git diff --stat origin/main...HEAD: one workflow file deleted, nothing else changed.
  • Checked that no other workflow in this repository references claude.yml.

Summary by CodeRabbit

  • Chores
    • Removed the automated workflow that ran in response to issue and pull request activity. Those events no longer trigger this workflow.
    • No changes were made to the application’s user-facing features.

CodeRabbit now reviews pull requests, so Claude no longer answers
@claude mentions in this repository.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: QUIET
  • Plan: Advanced
  • Run ID: cbf2027a-4939-436b-9b90-a4614c42be9d
Commits

Reviewing files that changed from the base of the PR and between 324cd44 and 9fca7ea.

Files selected for processing: 1
  • .github/workflows/claude.yml
Files with no reviewable changes: 1
  • .github/workflows/claude.yml

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 6 remain after this review.

Walkthrough

The pull request deletes .github/workflows/claude.yml. The workflow’s event triggers, reusable workflow invocation, inherited secrets, and instructions for reviewing generated /dist output are removed.

Changes: Claude workflow removal

Layer / File(s) Summary
Remove Claude workflow
.github/workflows/claude.yml
The workflow and its event triggers, reusable workflow invocation, inherited secrets, and review instructions are removed.

Priority: Low

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

Change: Other

Possibly related issues

  • simpleanalytics/dashboard#971 — This change removes the Claude workflow in this repository, but does not address the dashboard issue’s SOC 2 label concern.

Suggested labels: change: needs review

Suggested reviewers: adriaandotcom

Merge Risk: Minimal · up to 9fca7

The change turns off the requested Claude automation. No merge-blocking concern is identified.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: removing the @claude workflow.
Linked Issues check Passed Issue #145 requires disabling the workflow that runs Claude Code for @claude activity. The whole-PR diff deletes .github/workflows/claude.yml, including its issue_comment, `pull_request_review_c…
Out of Scope Changes check Passed The whole-PR diff contains only the deletion of .github/workflows/claude.yml. The change directly supports issue #145 and contains no unrelated source, test, or configuration changes.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Full details: Linked Issues check

Explanation

Issue #145 requires disabling the workflow that runs Claude Code for @claude activity. The whole-PR diff deletes .github/workflows/claude.yml, including its issue_comment, pull_request_review_comment, issues, and pull_request_review triggers and the reusable Claude workflow job. This implements the requested behavior.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.

Finishing Touches: 💡 1
🛠️ Fix failing CI checks: 💡
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

Claude still answers @claude mentions in scripts

1 participant