Skip to content

[auto] cat-mode: say it applies by default behind CATSTACK_CAT_MODE_DEFAULT - #248

Merged
mergify[bot] merged 1 commit into
mainfrom
stack/EdbertChan/hooks/cat-mode-default-20260907/auto-cat-mode-say-applies-default-behind-catstack--4a47f2e3
Sep 7, 2026
Merged

[auto] cat-mode: say it applies by default behind CATSTACK_CAT_MODE_DEFAULT#248
mergify[bot] merged 1 commit into
mainfrom
stack/EdbertChan/hooks/cat-mode-default-20260907/auto-cat-mode-say-applies-default-behind-catstack--4a47f2e3

Conversation

@EdbertChan

@EdbertChan EdbertChan commented Sep 7, 2026

Copy link
Copy Markdown
Owner

The cat-mode-default hook applies this skill on every investigation or
execution prompt when CATSTACK_CAT_MODE_DEFAULT=1. Name that in the skill
body so a reader of the skill knows why it applied without being typed,
and keep disable-model-invocation: true. Lock it with a test.

Co-Authored-By: Claude Fable 5.1 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_019CoNSA1EKhBHfVUX7feoKR


Note

Low Risk
Documentation and structural regression tests only; no hook or runtime behavior changes.

Overview
The cat-mode skill intro now states that conventions apply by default when CATSTACK_CAT_MODE_DEFAULT=1, injected by the cat-mode-default hook—so readers understand auto-application without a typed /cat-mode. Frontmatter disable-model-invocation: true is unchanged.

Tests add TestCatModeDefaultHookPointer to lock that prose, confirm engine/hooks/cat-mode-default exists, and keep disable-model-invocation true. cat-mode-default is allowlisted in the skill-reference checker so the hook name in backticks is not treated as a missing skill.

Reviewed by Cursor Bugbot for commit 14bd122. Bugbot is set up for automated code reviews on this repo. Configure here.

@EdbertChan

Copy link
Copy Markdown
Owner Author

This pull request is part of a Mergify stack:

# Pull Request Link
1 [auto] hook: cat-mode-default applies cat-mode on work turns behind an env flag #247
2 [auto] cat-mode: say it applies by default behind CATSTACK_CAT_MODE_DEFAULT #248 👈

@cursor

cursor Bot commented Sep 7, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_71e32fe3-2552-4d01-bfa8-fca6718af0af)

@EdbertChan
EdbertChan changed the base branch from stack/EdbertChan/hooks/cat-mode-default-20260907/auto-hook-cat-mode-default-applies-cat-mode-work--e353698b to main September 7, 2026 17:01
@EdbertChan

Copy link
Copy Markdown
Owner Author

@Mergifyio queue

@mergify

mergify Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

This pull request spent 5 minutes 47 seconds in the queue, including 4 minutes 57 seconds running CI.

Waiting for
  • check-success = test
All conditions
  • check-success = test
  • check-success = lint

Reason

The merge conditions cannot be satisfied due to failing checks

  • test

Failing checks:

Hint

You may have to fix your CI before adding the pull request to the queue again.
If you update this pull request, to fix the CI, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio queue comment.

Requeued — the merge queue status continues in this comment ↓.

@mergify mergify Bot added the queued label Sep 7, 2026
@mergify mergify Bot added dequeued and removed queued labels Sep 7, 2026
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/hooks/cat-mode-default-20260907/auto-cat-mode-say-applies-default-behind-catstack--4a47f2e3 branch from 23786ee to fa1a89d Compare September 7, 2026 17:13
@EdbertChan

EdbertChan commented Sep 7, 2026

Copy link
Copy Markdown
Owner Author

Revision history

# Type Changes Reason Date
1 initial 23786ee 2026-09-07 17:13 UTC
2 content 23786ee → fa1a89d 2026-09-07 17:13 UTC
3 content fa1a89d → 14bd122 (raw) 2026-09-07 17:13 UTC

@EdbertChan

Copy link
Copy Markdown
Owner Author

@Mergifyio queue

@cursor

cursor Bot commented Sep 7, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_b017912f-f9e3-426b-87f8-df08f22f61ff)

…EFAULT

The cat-mode-default hook applies this skill on every investigation or
execution prompt when CATSTACK_CAT_MODE_DEFAULT=1. Name that in the skill
body so a reader of the skill knows why it applied without being typed,
and keep disable-model-invocation: true. Lock it with a test.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019CoNSA1EKhBHfVUX7feoKR
Change-Id: I4a47f2e35504c4e848448965bf40db4971c933c5
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/hooks/cat-mode-default-20260907/auto-cat-mode-say-applies-default-behind-catstack--4a47f2e3 branch from fa1a89d to 14bd122 Compare September 7, 2026 17:13
@EdbertChan

Copy link
Copy Markdown
Owner Author

@Mergifyio queue

@cursor

cursor Bot commented Sep 7, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_6e768b1f-a889-4c78-8b92-4fb2f67c8b2c)

@mergify

mergify Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

queue

☑️ Command queue ignored because it is already running from a previous command.

@mergify

mergify Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

queue

🛑 The pull request has been removed from the queue admin-bypass

Details

The pull request #248 has been manually updated.

You can take a look at Mergify Merge Queue check runs for more details about the failure.

@mergify mergify Bot removed the dequeued label Sep 7, 2026
@EdbertChan

Copy link
Copy Markdown
Owner Author

@Mergifyio queue

@mergify

mergify Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

This pull request spent 7 minutes 24 seconds in the queue, including 6 minutes 26 seconds running CI.

Required conditions to merge
  • check-success = lint
  • check-success = test

@mergify mergify Bot added the queued label Sep 7, 2026
@mergify
mergify Bot merged commit 4589d5b into main Sep 7, 2026
6 checks passed
@mergify mergify Bot removed the queued label Sep 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant