Skip to content

fix: add OPERATIONS session type for PM/planning sessions (#2914)#2923

Merged
rysweet merged 4 commits intomainfrom
fix/2914-operations-session-type
Mar 7, 2026
Merged

fix: add OPERATIONS session type for PM/planning sessions (#2914)#2923
rysweet merged 4 commits intomainfrom
fix/2914-operations-session-type

Conversation

@rysweet
Copy link
Owner

@rysweet rysweet commented Mar 7, 2026

Summary

Root Cause

Power-steering incorrectly activated dev workflow checks on PM sessions (e.g. /pm-architect) because heavy Read/Grep tool usage triggered INVESTIGATION classification, which applies workflow_invocation, next_steps, and documentation_updates checks that are irrelevant for planning sessions.

Changes

  • session_detection.py: Add OPERATIONS_KEYWORDS list, detect before tool usage analysis
  • considerations.yaml: Document OPERATIONS session type in header
  • Both .claude/ and amplifier-bundle/ copies synced

Closes #2914, Closes #2913

Test plan

  • Run /pm-architect and verify no power-steering dev checks fire
  • Verify DEVELOPMENT sessions still get full checks
  • Verify env override AMPLIHACK_SESSION_TYPE=OPERATIONS works

Generated with Claude Code

Ubuntu and others added 2 commits March 7, 2026 21:35
Power-steering incorrectly activated dev checks on Q&A/PM sessions
(e.g. /pm-architect) because Read/Grep tool usage triggered
INVESTIGATION classification, which applies workflow checks.

Add OPERATIONS session type detected via PM/planning keywords
(prioritize, backlog, roadmap, sprint, triage, etc.) that skips
all power-steering considerations — same as SIMPLE sessions.

Detection priority: SIMPLE > OPERATIONS > DEVELOPMENT > INVESTIGATION.

Closes #2914, Closes #2913

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2026

🤖 Auto-fixed version bump

The version in pyproject.toml has been automatically bumped to the next patch version.

If you need a minor or major version bump instead, please update pyproject.toml manually and push the change.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2026

Repo Guardian - Passed

All changed files have been reviewed:

  • Configuration files (.yaml, .toml) ✅
  • Production Python code adding OPERATIONS session type feature ✅

No ephemeral content detected. All files are durable and appropriate for the repository.

AI generated by Repo Guardian

Ubuntu and others added 2 commits March 7, 2026 21:40
12 unit tests covering:
- PM/planning keyword detection (pm-architect, backlog, roadmap, sprint)
- OPERATIONS skips all considerations
- Env override AMPLIHACK_SESSION_TYPE=OPERATIONS
- Priority: SIMPLE > OPERATIONS > DEVELOPMENT > INVESTIGATION
- Code modifications with operations keywords

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
8 tests covering both Claude and Copilot sessions:
- Claude /pm-architect → OPERATIONS classification
- Claude full check() flow → approve (no blocks)
- Copilot backlog triage → OPERATIONS classification
- Copilot full check() flow → approve (no blocks)
- Development sessions still get full checks
- Simulates real transcript structure with tool_use blocks

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2026

Repo Guardian - Passed

All changed files have been reviewed:

Files Changed (8 total)

  • .claude/tools/amplihack/considerations.yaml - Production configuration ✅
  • .claude/tools/amplihack/hooks/power_steering_checker/session_detection.py - Production code ✅
  • .claude/tools/amplihack/hooks/tests/test_power_steering_operations_sessions.py - Test suite ✅
  • amplifier-bundle/tools/amplihack/considerations.yaml - Mirror config ✅
  • amplifier-bundle/tools/amplihack/hooks/power_steering_checker/session_detection.py - Mirror code ✅
  • amplifier-bundle/tools/amplihack/hooks/tests/test_power_steering_operations_sessions.py - Mirror tests ✅
  • pyproject.toml - Version bump (0.5.116) ✅
  • tests/outside_in/test_operations_session_e2e.py - Outside-in test suite ✅

Assessment

No ephemeral content detected. All files are:

  • Durable: Production code implementing OPERATIONS session type feature
  • Reusable: Test suites and configuration files for permanent use
  • Part of project structure: Follow established patterns (mirrored .claude/amplifier-bundle structure)

The changes add a permanent feature to classify PM/planning sessions as OPERATIONS type and skip development workflow checks. This is proper feature development, not temporary content.

AI generated by Repo Guardian

AI generated by Repo Guardian

@rysweet rysweet merged commit c7c5850 into main Mar 7, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant