oh-my-customcode 0.152.0 → 0.154.0

package/dist/cli/index.js

@@ -2334,7 +2334,7 @@ var init_package = __esm(() => {
     workspaces: [
       "packages/*"
     ],
-    version: "0.152.0",
+    version: "0.154.0",
     description: "Batteries-included agent harness for Claude Code",
     type: "module",
     bin: {

package/dist/index.js

@@ -2031,7 +2031,7 @@ var package_default = {
   workspaces: [
     "packages/*"
   ],
-  version: "0.152.0",
+  version: "0.154.0",
   description: "Batteries-included agent harness for Claude Code",
   type: "module",
   bin: {

package/package.json

@@ -3,7 +3,7 @@
   "workspaces": [
     "packages/*"
   ],
-  "version": "0.152.0",
+  "version": "0.154.0",
   "description": "Batteries-included agent harness for Claude Code",
   "type": "module",
   "bin": {

package/templates/.claude/hooks/hooks.json

@@ -208,6 +208,16 @@
         ],
         "description": "Advisory pre-processing of user input \u2014 skill matching hints and intent analysis"
       },
+      {
+        "matcher": "*",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "bash .claude/hooks/scripts/r007-r008-drift-advisor.sh"
+          }
+        ],
+        "description": "Proactive R007/R008 drift advisory \u2014 checks last assistant turn for identification omission (#1229)"
+      },
       {
         "matcher": "*",
         "hooks": [

package/templates/.claude/hooks/scripts/r007-r008-drift-advisor.sh

@@ -0,0 +1,117 @@
+#!/usr/bin/env bash
+# r007-r008-drift-advisor.sh — UserPromptSubmit hook: PROACTIVE R007/R008 drift advisory (#1229)
+#
+# Inspects the LAST completed assistant turn in the session transcript for R007/R008
+# compliance BEFORE Claude responds. If the previous turn drifted (missing identification
+# header / tool prefix), emits a stderr advisory so the upcoming response self-corrects.
+#
+# This is the PROACTIVE complement to the retroactive session-reflection.sh (Stop hook, #1190).
+# Detection patterns are reused from session-reflection.sh.
+#
+# Advisory-only: ALWAYS exits 0, ALWAYS passes stdin through to stdout, NEVER blocks.
+# Performance: parses ONLY the last assistant turn (not the whole transcript).
+#
+# 환경변수 override (테스트/디버깅용):
+#   OMCUSTOM_R007_ADVISOR=off  — advisory 완전 비활성화 (pass-through)
+#   OMCUSTOM_TRANSCRIPT_BASE   — transcript 디렉토리 경로 override
+
+set -euo pipefail
+
+# ── UserPromptSubmit 프로토콜: stdin을 먼저 읽음 ──
+input=$(cat)
+
+# ── Opt-out 체크 ──
+if [ "${OMCUSTOM_R007_ADVISOR:-}" = "off" ]; then
+  echo "$input"
+  exit 0
+fi
+
+# ── jq 의존성 체크 ──
+if ! command -v jq >/dev/null 2>&1; then
+  echo "$input"
+  exit 0
+fi
+
+# ── session_id 추출 ──
+session_id=$(echo "$input" | jq -r '.session_id // empty' 2>/dev/null)
+if [ -z "$session_id" ]; then
+  echo "$input"
+  exit 0
+fi
+
+# ── 경로 결정 (환경변수 override 지원) ──
+TRANSCRIPT_BASE="${OMCUSTOM_TRANSCRIPT_BASE:-${HOME}/.claude/projects/-Users-sangyi-workspace-projects-oh-my-customcode}"
+TRANSCRIPT_PATH="${TRANSCRIPT_BASE}/${session_id}.jsonl"
+
+if [ ! -f "$TRANSCRIPT_PATH" ]; then
+  echo "$input"
+  exit 0
+fi
+
+# ── 마지막 assistant 메시지 추출 (성능: 전체 transcript 스캔 회피) ──
+# 파일을 역순으로 읽으며 첫 번째 role=="assistant" 라인을 찾는다.
+last_assistant=""
+while IFS= read -r line; do
+  role=$(echo "$line" | jq -r '.role // empty' 2>/dev/null) || continue
+  if [ "$role" = "assistant" ]; then
+    last_assistant="$line"
+    break
+  fi
+done < <(tail -r "$TRANSCRIPT_PATH" 2>/dev/null || tac "$TRANSCRIPT_PATH" 2>/dev/null)
+
+if [ -z "$last_assistant" ]; then
+  echo "$input"
+  exit 0
+fi
+
+# ── content 배열 파싱 ──
+content_raw=$(echo "$last_assistant" | jq -c '.content // []' 2>/dev/null) || content_raw="[]"
+
+r007_violations=0
+r008_violations=0
+
+# ── R007: 첫 번째 text 블록의 첫 줄 체크 ──
+first_text=$(echo "$content_raw" | jq -r '[.[] | select(.type == "text")][0].text // empty' 2>/dev/null) || first_text=""
+if [ -n "$first_text" ]; then
+  first_line=$(printf '%s' "$first_text" | head -1)
+  # R007 패턴: '┌─ Agent:' 또는 '[anything]' 단축 형태
+  if ! printf '%s' "$first_line" | grep -qE '(^┌─ Agent:|^\[.+\])'; then
+    r007_violations=$((r007_violations + 1))
+  fi
+fi
+
+# ── R008: tool_use 블록 직전 text에 prefix 체크 ──
+content_length=$(echo "$content_raw" | jq 'length' 2>/dev/null) || content_length=0
+i=0
+while [ "$i" -lt "$content_length" ]; do
+  block_type=$(echo "$content_raw" | jq -r ".[$i].type // empty" 2>/dev/null) || { i=$((i+1)); continue; }
+
+  if [ "$block_type" = "tool_use" ]; then
+    has_prefix=false
+    if [ "$i" -gt 0 ]; then
+      prev_type=$(echo "$content_raw" | jq -r ".[$(( i - 1 ))].type // empty" 2>/dev/null) || true
+      if [ "$prev_type" = "text" ]; then
+        prev_text=$(echo "$content_raw" | jq -r ".[$(( i - 1 ))].text // empty" 2>/dev/null) || true
+        # R008 패턴: '[agent-name][model] → Tool:' 또는 '→ Target:'
+        if printf '%s' "$prev_text" | grep -qE '\[.+\]\[.+\] ?(→|->|—>) ?(Tool|Target):'; then
+          has_prefix=true
+        fi
+      fi
+    fi
+    if [ "$has_prefix" = "false" ]; then
+      r008_violations=$((r008_violations + 1))
+    fi
+  fi
+
+  i=$((i+1))
+done
+
+# ── advisory 출력 (위반 시에만) ──
+if [ "$r007_violations" -gt 0 ] || [ "$r008_violations" -gt 0 ]; then
+  printf '[R007/R008 Advisory] 직전 응답에서 식별 누락 감지 (R007 헤더=%d, R008 접두사=%d). 이번 응답은 ┌─ Agent: 헤더로 시작하고, 모든 도구 호출에 [agent][model] → Tool: 접두사를 포함하십시오.\n' \
+    "$r007_violations" "$r008_violations" >&2
+fi
+
+# ── 항상 stdin pass-through 후 exit 0 (advisory는 절대 차단 금지) ──
+echo "$input"
+exit 0

package/templates/.claude/rules/MUST-intent-transparency.md

@@ -103,6 +103,25 @@ Reference issues: #1188 item #4.
 
 **Why**: 사용자 directive 일관성 — #1208 보고. 같은 세션 내 동일 의도를 반복 차단하면 R015 user directive persistence 위반.
 
+### Destructive Operation Approval Persistence (Generalized)
+
+The Git Push Continuation pattern (first-time strict / follow-up relaxed, scoped to session + category + target) generalizes to ALL repeated destructive operations within the same session. Examples: `supabase db push`, `terraform apply`, `kubectl delete`, bulk file deletes, database migrations.
+
+**Scope**: once the user explicitly approves a destructive operation of category C against target T in a session, follow-up operations of the SAME C + SAME T do NOT require re-confirmation. An advisory warning is still emitted. A different category or different target always requires fresh confirmation.
+
+| Scenario | Behavior |
+|----------|----------|
+| 1st explicit approval (category C, target T) | Proceed; advisory warning emitted |
+| Follow-up same session (same C + same T) | No re-confirmation (directive persistence) |
+| Different category or target | Fresh confirmation required |
+| Platform classifier still prompts | Advise user: add `settings.json` permission rule for the specific command |
+
+**R001 exclusion (MUST)**: R001-listed catastrophic git operations (`git reset --hard`, `git clean -fd`, `git push --force` to shared branches, `git branch -D` with unmerged commits) are EXCLUDED from this persistence rule — they always require explicit per-invocation approval regardless of prior session approvals.
+
+**Boundary / honesty note**: This rule is ADVISORY and governs model behavior only. It CANNOT suppress Claude Code's platform-level auto-mode classifier prompts. For genuine prompt suppression on a repeated destructive command, the user must add a `settings.json` permission allow rule scoped to the specific command (e.g., a specific `supabase db push` invocation). The model SHOULD surface this workaround when the user expresses friction about repeated prompts.
+
+Cross-references: R001 (safety — destructive operation pre-checks still apply), R002 (permission tiers). Reference issues: #1230, #1226 (item 2).
+
 ## Agent Triggers
 
 Defined in `.claude/skills/intent-detection/patterns/agent-triggers.yaml`. Each agent has keywords, file patterns, actions, and base confidence.

package/templates/.claude/rules/SHOULD-memory-integration.md

@@ -351,7 +351,7 @@ MCP tools (claude-mem, episodic-memory) are **orchestrator-scoped** and not inhe
 
 ### Session-End Self-Check (MANDATORY)
 
-(1) sys-memory-keeper updated MEMORY.md? (2) claude-mem save attempted? (3) If `omcustom-feedback` skill is active, prompt user to trigger it? All three required before confirming to user. See full self-check via Read tool.
+(1) sys-memory-keeper updated MEMORY.md? (2) claude-mem save attempted? (3) If `omcustom-feedback` skill is active, model MAY draft a retrospective feedback issue for user approval — or prompt user to trigger it manually. All three required before confirming to user. See full self-check via Read tool.
 
 <!-- DETAIL: Session-End Self-Check (MANDATORY)
 ```
@@ -367,8 +367,10 @@ MCP tools (claude-mem, episodic-memory) are **orchestrator-scoped** and not inhe
 ║     NO  → ToolSearch + save now                                  ║
 ║                                                                   ║
 ║  3. Is omcustom-feedback skill available in this project?        ║
-║     YES → Ask user: "이번 세션 피드백을 omcustom-feedback로     ║
-║          기록하시겠습니까?" — accept skip                        ║
+║     YES → If notable friction/learning observed: MODEL DRAFTS    ║
+║          retrospective issue → presents via Phase 4A preview     ║
+║          gate for user approval. Otherwise: prompt user to       ║
+║          trigger manually. Accept skip either way.               ║
 ║     NO  → Skip                                                    ║
 ║                                                                   ║
 ║  Note: episodic-memory auto-indexes conversations after session  ║
@@ -381,6 +383,33 @@ MCP tools (claude-mem, episodic-memory) are **orchestrator-scoped** and not inhe
 ```
 -->
 
+### Session-End Retrospective Feedback (Model-Drafted)
+
+Since `omcustom-feedback` is now model-invocable (#1227), the model MAY draft a retrospective feedback issue at session end — instead of only prompting the user to compose one manually.
+
+**Workflow**
+
+1. Model detects notable friction, workarounds, or harness gaps observed during the session.
+2. Model drafts a feedback issue (title + body) using the `omcustom-feedback` skill.
+3. Draft is presented through the skill's **Phase 4A preview + confirmation gate**. The user reviews and approves before any GitHub issue is created.
+4. The model NEVER auto-submits. User approval is always required.
+
+**Trigger conditions** (all must be true):
+- Session-end detected
+- Notable friction or learning observed during the session
+- `omcustom-feedback` skill active in this project
+
+**Distinction from manual path**
+
+| Path | Who drafts | Who approves | When |
+|------|-----------|--------------|------|
+| Manual (existing) | User | User | User chooses to file feedback |
+| Model-drafted (new, #1226 item 3) | Model | User (Phase 4A gate) | Session-end with notable friction |
+
+The model-drafted path is an enhancement: it proposes a concrete draft rather than asking the user to compose from scratch. Both paths remain valid; neither replaces the other.
+
+References: #1226 (item 3), #1227.
+
 ### Failure Policy
 
 - MCP saves are **non-blocking**: memory failure MUST NOT prevent session from ending
@@ -418,7 +447,7 @@ Phase 1 COEXIST 기간 중 세션 종료 시:
 1. sys-memory-keeper가 MEMORY.md 갱신? → YES: 계속
 2. claude-mem 저장 시도? → YES (기존 항목)
 3. AgentMemory 저장 시도? → YES (COEXIST 추가)
-4. omcustom-feedback 권유? → YES (활성 시) / 스킵 (비활성 시)
+4. omcustom-feedback 처리? → YES (활성 시, notable friction 있으면 model draft → Phase 4A gate; 없으면 사용자 권유) / 스킵 (비활성 시)
 네 단계 모두 완료 후 사용자에게 확인. 둘 중 하나 실패해도 비차단.
 
 ### Phase 2 진입 전 필수 조건

package/templates/manifest.json

@@ -1,5 +1,5 @@
 {
-  "version": "0.152.0",
+  "version": "0.154.0",
   "lastUpdated": "2026-05-20T00:00:00.000Z",
   "omcustomMinClaudeCode": "2.1.121",
   "omcustomMinClaudeCodeReason": "Sensitive-path direct Write/Edit on .claude/** under bypassPermissions (R010 deprecation, #1101)",