@scotthamilton77/sidekick 0.1.2 → 0.1.4

package/assets/sidekick/defaults/features/reminders.defaults.yaml

@@ -63,6 +63,262 @@ settings:
   # -1 = unlimited (default), 0 = disabled, positive = exact limit
   max_verification_cycles: -1
 
+  # Per-tool verification tracking.
+  # Each tool category tracks whether its verification command has been run.
+  # When source files are edited, tools transition to STAGED (needing re-run).
+  # When the tool's command is observed, it transitions to VERIFIED.
+  # After VERIFIED, a configurable number of file edits (clearing_threshold) triggers re-staging.
+  #
+  # Pattern fields:
+  #   tool_id: stable key for override/removal by project or user config
+  #   tool: token pattern for matching (null = disabled)
+  #   scope: metadata — project, package, or file
+  verification_tools:
+    build:
+      enabled: true
+      patterns:
+        # TypeScript/JavaScript
+        - tool_id: tsc
+          tool: "tsc"
+          scope: project
+        - tool_id: esbuild
+          tool: "esbuild"
+          scope: file
+        - tool_id: pnpm-filter-build
+          tool: "pnpm --filter * build"
+          scope: package
+        - tool_id: pnpm-build
+          tool: "pnpm build"
+          scope: project
+        - tool_id: npm-build
+          tool: "npm run build"
+          scope: project
+        - tool_id: yarn-workspace-build
+          tool: "yarn workspace * build"
+          scope: package
+        - tool_id: yarn-build
+          tool: "yarn build"
+          scope: project
+        # Python
+        - tool_id: python-setup-build
+          tool: "python setup.py build"
+          scope: project
+        - tool_id: pip-install
+          tool: "pip install"
+          scope: project
+        - tool_id: poetry-build
+          tool: "poetry build"
+          scope: project
+        # JVM
+        - tool_id: mvn-compile
+          tool: "mvn compile"
+          scope: project
+        - tool_id: mvn-package
+          tool: "mvn package"
+          scope: project
+        - tool_id: gradle-build
+          tool: "gradle build"
+          scope: project
+        - tool_id: gradlew-build
+          tool: "./gradlew build"
+          scope: project
+        # Go
+        - tool_id: go-build
+          tool: "go build"
+          scope: project
+        # Rust
+        - tool_id: cargo-build
+          tool: "cargo build"
+          scope: project
+        # C/C++
+        - tool_id: make-build
+          tool: "make build"
+          scope: project
+        - tool_id: make-default
+          tool: "make"
+          scope: project
+        - tool_id: cmake-build
+          tool: "cmake --build"
+          scope: project
+        # Containers
+        - tool_id: docker-build
+          tool: "docker build"
+          scope: project
+      clearing_threshold: 3
+      clearing_patterns:
+        - "**/*.ts"
+        - "**/*.tsx"
+        - "**/*.js"
+        - "**/*.jsx"
+        - "**/*.py"
+        - "**/*.java"
+        - "**/*.kt"
+        - "**/*.go"
+        - "**/*.rs"
+        - "**/*.c"
+        - "**/*.cpp"
+        - "**/*.cs"
+    typecheck:
+      enabled: true
+      patterns:
+        - tool_id: tsc-noEmit
+          tool: "tsc --noEmit"
+          scope: project
+        - tool_id: pnpm-filter-typecheck
+          tool: "pnpm --filter * typecheck"
+          scope: package
+        - tool_id: pnpm-typecheck
+          tool: "pnpm typecheck"
+          scope: project
+        - tool_id: npm-typecheck
+          tool: "npm run typecheck"
+          scope: project
+        - tool_id: yarn-workspace-typecheck
+          tool: "yarn workspace * typecheck"
+          scope: package
+        - tool_id: yarn-typecheck
+          tool: "yarn typecheck"
+          scope: project
+        - tool_id: mypy
+          tool: "mypy"
+          scope: project
+        - tool_id: pyright
+          tool: "pyright"
+          scope: project
+        - tool_id: pytype
+          tool: "pytype"
+          scope: project
+        - tool_id: go-vet
+          tool: "go vet"
+          scope: project
+      clearing_threshold: 3
+      clearing_patterns:
+        - "**/*.ts"
+        - "**/*.tsx"
+        - "**/*.py"
+        - "**/*.go"
+    test:
+      enabled: true
+      patterns:
+        - tool_id: vitest
+          tool: "vitest"
+          scope: project
+        - tool_id: jest
+          tool: "jest"
+          scope: project
+        - tool_id: pnpm-filter-test
+          tool: "pnpm --filter * test"
+          scope: package
+        - tool_id: pnpm-test
+          tool: "pnpm test"
+          scope: project
+        - tool_id: npm-test
+          tool: "npm test"
+          scope: project
+        - tool_id: yarn-workspace-test
+          tool: "yarn workspace * test"
+          scope: package
+        - tool_id: yarn-test
+          tool: "yarn test"
+          scope: project
+        - tool_id: pytest
+          tool: "pytest"
+          scope: project
+        - tool_id: python-pytest
+          tool: "python -m pytest"
+          scope: project
+        - tool_id: python-unittest
+          tool: "python -m unittest"
+          scope: project
+        - tool_id: mvn-test
+          tool: "mvn test"
+          scope: project
+        - tool_id: gradle-test
+          tool: "gradle test"
+          scope: project
+        - tool_id: gradlew-test
+          tool: "./gradlew test"
+          scope: project
+        - tool_id: go-test
+          tool: "go test"
+          scope: project
+        - tool_id: cargo-test
+          tool: "cargo test"
+          scope: project
+        - tool_id: dotnet-test
+          tool: "dotnet test"
+          scope: project
+        - tool_id: make-test
+          tool: "make test"
+          scope: project
+      clearing_threshold: 3
+      clearing_patterns:
+        - "**/*.ts"
+        - "**/*.tsx"
+        - "**/*.js"
+        - "**/*.jsx"
+        - "**/*.py"
+        - "**/*.java"
+        - "**/*.kt"
+        - "**/*.go"
+        - "**/*.rs"
+        - "**/*.test.*"
+        - "**/*.spec.*"
+        - "**/test_*"
+    lint:
+      enabled: true
+      patterns:
+        - tool_id: eslint
+          tool: "eslint"
+          scope: project
+        - tool_id: pnpm-filter-lint
+          tool: "pnpm --filter * lint"
+          scope: package
+        - tool_id: pnpm-lint
+          tool: "pnpm lint"
+          scope: project
+        - tool_id: npm-lint
+          tool: "npm run lint"
+          scope: project
+        - tool_id: yarn-workspace-lint
+          tool: "yarn workspace * lint"
+          scope: package
+        - tool_id: yarn-lint
+          tool: "yarn lint"
+          scope: project
+        - tool_id: ruff
+          tool: "ruff"
+          scope: project
+        - tool_id: flake8
+          tool: "flake8"
+          scope: project
+        - tool_id: pylint
+          tool: "pylint"
+          scope: project
+        - tool_id: golangci-lint
+          tool: "golangci-lint"
+          scope: project
+        - tool_id: cargo-clippy
+          tool: "cargo clippy"
+          scope: project
+        - tool_id: ktlint
+          tool: "ktlint"
+          scope: project
+        - tool_id: dotnet-format
+          tool: "dotnet format"
+          scope: project
+      clearing_threshold: 5
+      clearing_patterns:
+        - "**/*.ts"
+        - "**/*.tsx"
+        - "**/*.js"
+        - "**/*.jsx"
+        - "**/*.py"
+        - "**/*.java"
+        - "**/*.kt"
+        - "**/*.go"
+        - "**/*.rs"
+
   # Smart completion detection settings
   # Classifies assistant's stopping intent to avoid blocking when not claiming completion
   completion_detection:

package/assets/sidekick/prompts/session-summary.prompt.txt

@@ -126,6 +126,44 @@ Users often specify both a methodology and a goal. The methodology is NOT the ti
     If removing these leaves nothing meaningful, dig deeper into the transcript for the actual goal.
   </heuristic>
 </outcome_vs_approach>
+<title_stability>
+CRITICAL: The session title is an ANCHOR, not a weather vane.
+
+When previousConfidence is >0.8, the title has been validated. It should only change
+if the user genuinely pivots to a different task — not because they issued a
+procedural or mechanical instruction.
+
+  <procedural_instructions>
+    These are session-ending mechanics, NOT session-defining work:
+    - Git operations: commit, push, pull, rebase, merge, stash, checkout, branch
+    - PR/review: create PR, open PR, merge PR, request review, approve
+    - Deployment: deploy, publish, release, push to remote
+    - Housekeeping: clean up, close issues, update status, run linters
+
+    When the most recent user message is procedural:
+    - session_title: KEEP the established title (do not change)
+    - latest_intent: Freely reflect the procedural action
+    - pivot_detected: false
+  </procedural_instructions>
+
+  <stability_bias>
+    Title change threshold scales with confidence:
+    - previousConfidence >0.8: Only change on genuine task pivot (new domain, new goal)
+    - previousConfidence 0.6-0.8: Refine freely (more specific on same topic),
+      but only pivot if the new direction is clearly unrelated
+    - previousConfidence <=0.5: Change freely — direction is still forming
+
+    Ask: "If someone reviewed this session tomorrow, would they say the user
+    switched projects — or just finished up their current one?"
+  </stability_bias>
+
+  <edge_cases>
+    - "Set up CI/CD pipeline" after working on a feature -> IS a pivot (new goal)
+    - "Push it" after working on a feature -> NOT a pivot (finishing current work)
+    - "Now let's work on the auth bug" -> IS a pivot (explicitly new task)
+    - "Create a PR for this" -> NOT a pivot (shipping current work)
+  </edge_cases>
+</title_stability>
 </session_title_rules>
 
 <latest_intent_rules>

package/assets/sidekick/reminders/vc-build.yaml

@@ -0,0 +1,15 @@
+# vc-build.yaml
+# Per-tool verification reminder: build step
+id: vc-build
+blocking: true
+priority: 50
+persistent: false
+
+additionalContext: |
+  <vc-build>
+  You have modified source files but have not run a build step.
+  Run the project's build command before claiming completion.
+  </vc-build>
+
+userMessage: "Verification needed: build not run since last code changes"
+reason: "Source files modified without subsequent build verification"

package/assets/sidekick/reminders/vc-lint.yaml

@@ -0,0 +1,15 @@
+# vc-lint.yaml
+# Per-tool verification reminder: linting
+id: vc-lint
+blocking: true
+priority: 50
+persistent: false
+
+additionalContext: |
+  <vc-lint>
+  You have modified source files but have not run the linter.
+  Run the project's lint command before claiming completion.
+  </vc-lint>
+
+userMessage: "Verification needed: lint not run since last code changes"
+reason: "Source files modified without subsequent lint verification"

package/assets/sidekick/reminders/vc-test.yaml

@@ -0,0 +1,15 @@
+# vc-test.yaml
+# Per-tool verification reminder: test suite
+id: vc-test
+blocking: true
+priority: 50
+persistent: false
+
+additionalContext: |
+  <vc-test>
+  You have modified source files but have not run tests.
+  Run the relevant test suite before claiming completion.
+  </vc-test>
+
+userMessage: "Verification needed: tests not run since last code changes"
+reason: "Source files modified without subsequent test verification"

package/assets/sidekick/reminders/vc-typecheck.yaml

@@ -0,0 +1,15 @@
+# vc-typecheck.yaml
+# Per-tool verification reminder: type checking
+id: vc-typecheck
+blocking: true
+priority: 50
+persistent: false
+
+additionalContext: |
+  <vc-typecheck>
+  You have modified source files but have not run a type check.
+  Run the project's typecheck command before claiming completion.
+  </vc-typecheck>
+
+userMessage: "Verification needed: typecheck not run since last code changes"
+reason: "Source files modified without subsequent typecheck verification"

package/assets/sidekick/reminders/verify-completion.yaml

@@ -1,8 +1,9 @@
 # verify-completion.yaml
-# Blocks execution before task completion to enforce verification
+# Wrapper reminder that aggregates per-tool VC reminders.
+# Priority 51 ensures this is primary (consumed first); per-tool reminders at 50 are secondaries.
 id: verify-completion
 blocking: true
-priority: 50
+priority: 51
 persistent: false
 
 additionalContext: |
@@ -14,24 +15,8 @@ additionalContext: |
   - The verification evidence is done just before this reminder was triggered.
   - You're not claiming "done" but stopping to give an update or ask for more info.
 
-  Otherwise, this step is mandatory to ensure quality and completeness. Make sure you're really complete, then provide a revised report to the user.
-
-  For significant (more than just a few lines) code changes (only if not already done recently):
-  - Run build: `pnpm build` (excludes test files - both required)
-  - Run relevant tests: `pnpm test` (targeted tests for changed code)
-  - Run the `code-simplifier` agent to simplify and refine the code for clarity and maintainability.
-  - Run the `code-reviewer` agent to review your changes against the plan and coding standards.
-  - If your changes impact existing documentation that you haven't already updated,
-    inform the user and ask whether this should be updated as a next step.
-
-  General:
-  - Confirm all requirements from the original request are met
-  - Check for unintended side effects or regressions
-  - Ensure no temporary files, debug code, or stray changes remain
-  - DON'T COMMIT unless explicitly instructed. User says "make changes" ≠ "commit changes". Wait for explicit approval.
+  Otherwise, this step is mandatory. The following verification steps are outstanding:
   </completion-verification-required>
-  For Code Changes:
 
 userMessage: "Asking the agent to verify completion before stopping..."
-
-reason: "Verify completion before stopping - did you run tests and checks?"
+reason: "Verify completion before stopping - outstanding verification steps detected"