npm - claude-dev-env - Versions diffs - 2.15.1 → 2.17.0 - Mend

claude-dev-env 2.15.1 → 2.17.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (841) hide show

package/.agents/skills/shared-extraction-audit/SKILL.md ADDED Viewed

@@ -0,0 +1,160 @@
+---
+name: shared-extraction-audit
+description: Audits pull requests and packages for general helpers living in workflow-specific code instead of shared_utils, then applies extraction fixes by default in small tested CLs. Use when reviewing PRs for architectural layering, migrating utilities out of cert_fix_queue/theme_dialer_pipeline/skills, auditing "similar offenses", or when the user mentions extraction audit, shared_utils migration, orchestration-only packages, thin wrappers, or layer inversions.
+---
+# Shared Extraction Audit
+Audit PRs and packages, report findings, and apply extraction fixes by default to keep **orchestration packages thin** and **shared_utils canonical**.
+## When to use
+- User points at a PR (e.g. `#1954`, `#1965`) and asks for the same review loop
+- A package grew `fix_one`, `_default_*`, backup upload, rembg session, or residual gates alongside orchestration
+- Before merge: confirm consumers are adapters, not second implementations
+## Target architecture
+| Layer | Holds | Examples |
+|-------|--------|----------|
+| **shared_utils** | Reusable actions, I/O, DB flips, inference, constants used by 2+ workflows | `status_promotion`, `background_removal`, `drive_production_backup`, `stp_filenames` |
+| **Workflow package** | Orchestration, CLI, sweep loops, workflow-specific constants only | `cert_fix_queue/pipeline/queue_run.py`, `theme_dialer_pipeline` compose runners |
+| **Skills / scripts** | Operator entrypoints that **delegate** to shared_utils | No parallel rembg/session stacks |
+**Load-bearing rule:** If another package would import it, put it in `shared_utils`. After the move, every former caller imports the shared symbol and the old module is deleted.
+## Audit workflow
+Copy to TodoWrite:
+```
+Shared extraction audit:
+- [ ] 1. Scope the PR (base branch, changed packages, stated goal)
+- [ ] 2. Map canonical homes already in shared_utils
+- [ ] 3. Grep for offense patterns (see reference/offense-taxonomy.md)
+- [ ] 4. Write prioritized findings (P0–P3)
+- [ ] 5. Apply the prioritized fix band by default; an explicit audit-only request ends after the report
+- [ ] 6. Extract in small CLs (~100 lines) + move/adjust tests
+- [ ] 7. Run scoped pytest, commit, push, update PR
+```
+### Step 1 — Scope
+```bash
+git fetch origin pull/<N>/head:pr-<N>   # or checkout feature branch
+git diff <base>...<head> --stat
+git diff <base>...<head> --name-only | rg 'shared_utils|cert_fix_queue|theme_dialer|clean_room|\.claude/skills'
+```
+Read the PR summary, then verify **what landed correctly** before listing offenses.
+### Step 2 — Search patterns
+Run these greps on the PR diff scope:
+```bash
+# Rename-only seams and re-export facades
+rg '_default_|def _.*_flipper|def _.*_one\b|promote_one|fix_one|backup_upload' <package> shared_utils
+rg '^from shared_utils\..* import \w+$' <package>
+# Layer inversions (shared importing consumer)
+rg 'from (cert_fix_queue|theme_dialer_pipeline|clean_room_theme_icons)\.' shared_utils/
+# Parallel stacks (second session/cache for same concern)
+rg 'new_session|lru_cache.*session|rembg\.remove' .claude/skills shared_utils
+# Duplicate constants
+rg 'ALPHA_CHANNEL_INDEX|RGBA_ALPHA|DEFAULT_.*_PREFIX|ONEUI_STP_BASENAME' --glob '*.py'
+# Duplicate test support
+rg 'fail_if_called|build_rejected_theme_row|preserve_supplied_alpha' --glob '**/tests/**'
+```
+### Step 3 — Classify offenses
+Use [reference/offense-taxonomy.md](reference/offense-taxonomy.md). Assign priority:
+| Priority | Meaning |
+|----------|---------|
+| **P0** | Layer inversion or duplicate production stack (must fix before merge) |
+| **P1** | General helper/constants/tests still in consumer package |
+| **P2** | Module shape, I/O overlap, monolith size |
+| **P3** | Stale names, docstrings, test attribute bugs |
+### Step 4 — Report template
+```markdown
+## PR #<N> extraction audit
+### Done well
+- [canonical modules and deletions]
+### P0 — …
+| Offense | Where | Target module | Tests |
+|---------|-------|---------------|-------|
+### Suggested fix order
+1. …
+```
+Apply the prioritized fixes after the report by default. An explicit audit-only request ends after the report.
+## Fix workflow
+### Extraction rules
+1. **Move behavior** — delete the consumer copy; every caller imports the shared symbol
+2. **Wire shared APIs at the call site** — e.g. `db_flipper=promote_theme_to_status` with `filename_prefix=...`
+3. **Retarget every caller, then delete the old module** — grep the old import path; remove re-export facades
+4. **Constants:** one source; workflow CLI keeps only workflow-specific names (`LEDGER_FILENAME`, exit codes)
+5. **Tests move with behavior** — shared module gets shared tests; orchestration keeps integration tests only
+6. **Small CLs** — one concern per commit (~100 lines); run scoped pytest after each
+### Canonical target map (this repo)
+| Need | Home |
+|------|------|
+| Theme DB status flip | `shared_utils/theme_db/status_promotion.py` |
+| Account folder ↔ ThemeAccount | `shared_utils/theme_persistence/account_adapter.py` |
+| STP basename parsing | `shared_utils/samsung_utils/stp_filenames.py` |
+| Drive backup upload | `shared_utils/files/drive_production_backup.py` |
+| Timestamp scratch dirs | `shared_utils/files/timestamp_directory_retention.py` |
+| Foreground extraction | `shared_utils/theme_assets/background_removal.py` |
+| Cert fix / promote actions | `shared_utils/theme_assets/cert_closeout/` |
+| Fixed-STP promotion sweep | `shared_utils/samsung_utils/promote_fixed_stp_sweep.py` |
+| Cert queue orchestration only | `cert_fix_queue/pipeline/` (`queue_run`) and `run_cli` |
+See also `.claude/rules/reuse-existing-tooling.md` before adding helpers.
+### Verification
+```bash
+# Package under change (from its directory or shared_utils)
+cd shared_utils && python -m pytest path/to/tests -n 0 -q
+cd cert_fix_queue && python -m pytest tests/ -n 0 -q
+# After moving tests: bump CI collect floor in .github/workflows/standalone-packages-tests.yml if applicable
+```
+Provide evidence: pytest output counts, not "should work".
+### Git / PR
+- Branch: `cursor/<topic>-593d` or existing PR branch
+- Commit message: `Extract <what> from <package> into shared_utils`
+- Update PR body: architecture table + test counts
+- Do not merge; draft PR unless user says otherwise
+## Anti-patterns (flag these)
+- `_default_*` adapter around a shared_utils function in a consumer module
+- Workflow module that only does `from shared_utils... import foo as foo`
+- `fix_one.py` / `promote_one.py` / `promote_run.py` / `backup_upload.py` in orchestration packages
+- `shared_utils` importing `theme_dialer_pipeline` or `cert_fix_queue`
+- Second rembg/BiRefNet session in a skill while `background_removal` exists
+- Identical `tests/support.py` in two packages
+- `config/constants.py` that only aliases extracted helpers
+## Examples
+See [reference/examples.md](reference/examples.md) for cert_fix_queue (#1965) and background_removal (#1954) audits.

package/.agents/skills/shared-extraction-audit/reference/examples.md ADDED Viewed

@@ -0,0 +1,46 @@
+# Audit examples (this repo)
+## cert_fix_queue — PR #1965
+**Goal:** `cert_fix_queue` holds orchestration only; general cert-fix utilities in `shared_utils`.
+**Done well:**
+- `locate_cert_folder_for_row`, `select_production_stp`, atomic copy, drive staged upload → shared_utils
+- Deleted `fix_one.py`, `promote_one.py`, `backup_upload.py` from pipeline
+**Offenses found and fixed:**
+- `_default_db_flipper` in `fix_theme.py` → wire `db_flipper=promote_theme_to_status` at the call site; delete the wrapper
+- `production_backup.py` under cert_closeout → `shared_utils/files/drive_production_backup.py`
+- Duplicate test `support.py` → `shared_utils/theme_assets/testing/cert_fix_test_support.py`
+- `fix_queue_constants.py` misnamed → `fix_outcomes.py`
+- `_account_by_fixed_stps_folder` → `cert_failure_folder_to_theme_account_map()` in `account_adapter`
+- `_fresh_run_directories` / scratch helpers → `allocate_timestamp_run_directory()`
+**Pipeline after:**
+```
+cert_fix_queue/pipeline/
+  queue_run.py
+  promote_run.py
+run_cli.py
+```
+---
+## background_removal — PR #1954
+**Goal:** One BiRefNet CPU path in `shared_utils/theme_assets/background_removal.py`.
+**Done well:**
+- Deleted `clean_room_theme_icons/chroma.py`, dialer `art_masks.py`, `png_io.py`
+- Added `pixel_masks.py`, `png_canonical.py`
+- Clean-room and dialer call `extract_foreground` / `extract_foreground_file`
+**Remaining offenses (audit):**
+- P0: Residual gates in `theme_dialer_pipeline/pipeline/chroma_extract.py` (shared algorithm output)
+- P0: `_extract_component_foreground` wrapper; same pattern inlined in `compose_from_masters.py`
+- P0: Parallel rembg in `misc-asset-remaster/scripts/generated_alpha_mask.py`
+- P1: Duplicate extraction test fixtures (dialer conftest vs clean-room conftest)
+- P1: `ALPHA_CHANNEL_INDEX` duplicated across dialer/clean-room/color_constants
+- P2: Atomic write helpers inside 723-line `background_removal.py`; PNG path vs `png_canonical`
+**Suggested fix order:** residual gates → shared test fixtures → dedupe constants → migrate misc rembg → I/O split (optional).

package/.agents/skills/shared-extraction-audit/reference/offense-taxonomy.md ADDED Viewed

@@ -0,0 +1,100 @@
+# Offense taxonomy
+## O1 — General action in workflow package
+**Signal:** Per-item function named `*_one`, `fix_*`, `promote_*`, `upload_*` in orchestration tree.
+**Example:** `cert_fix_queue/pipeline/fix_one.py` → `shared_utils/.../fix_theme.py` (`fix_cert_rejected_theme`).
+**Fix:** Move module; consumer passes workflow-specific `filename_prefix` / folder ids at call site.
+---
+## O2 — Thin wrapper / default seam / re-export facade
+**Signal:** `_default_db_flipper`, `_default_*` that only renames kwargs and calls shared_utils. Also: a workflow module that only re-exports a shared symbol under the old path.
+**Example:** Wrapper around `promote_theme_to_status` with `db_flip_log_filename_prefix`. After a move, `clean_room_theme_icons.rebuild_nine_patch` still contains `from shared_utils... import rebuild_nine_patch`.
+**Fix:** Update every caller to the shared symbol (`filename_prefix`, `shared_utils.theme_assets.nine_patch_rebuild`, etc.) and delete the wrapper or facade.
+---
+## O3 — Layer inversion
+**Signal:** `shared_utils/...` imports from `cert_fix_queue`, `theme_dialer_pipeline`, `clean_room_theme_icons`, or `.claude/skills/...`.
+**Fix:** Move imported symbol into shared_utils; consumer imports shared.
+---
+## O4 — Parallel production stack
+**Signal:** Second implementation of same concern (rembg session, despill, STP walk, DB flip) in skill or sibling package.
+**Example:** `misc-asset-remaster/.../generated_alpha_mask.py` vs `background_removal.py`.
+**Fix:** Delegate to canonical module; delete duplicate constants and session cache.
+---
+## O5 — Validation/gates on shared output in consumer
+**Signal:** Functions that only inspect output of shared algorithm (residual alpha, edge green excess) live in dialer/clean-room.
+**Example:** `theme_dialer_pipeline/pipeline/chroma_extract.py` using `edge_green_excess_mask` from shared.
+**Fix:** `shared_utils/theme_assets/background_removal/residual_gates.py` (or sibling); optional `extract_foreground_gated`.
+---
+## O6 — Duplicate constants
+**Signal:** Same literal or semantic constant in workflow `config/constants.py` and shared config.
+**Examples:** `ALPHA_CHANNEL_INDEX`, `ONEUI_STP_BASENAME_RE`, two `DEFAULT_*_DB_FLIP_*` with same value.
+**Fix:** Single definition in shared config; delete consumer copies.
+---
+## O7 — Misplaced config module
+**Signal:** Constants named for workflow A live under package B's config (e.g. `fix_queue_constants` under `cert_closeout`).
+**Fix:** Rename/move to owning workflow or shared outcomes module; drop re-export-only facades.
+---
+## O8 — Duplicate test support
+**Signal:** Identical or near-identical `tests/support.py`, conftest bypass fixtures, synthetic PNG writers in multiple packages.
+**Fix:** `shared_utils/theme_assets/testing/` or `shared_utils/.../tests/fixtures.py`; one import path.
+---
+## O9 — General I/O in domain module
+**Signal:** Atomic staged writes, Drive upload, UTC helpers duplicated inside a 700-line domain file.
+**Fix:** `shared_utils/files/` (e.g. `drive_production_backup`, `timestamp_directory_retention`, `record_persist`).
+---
+## O10 — Stale identifiers
+**Signal:** Docstrings/tests still say `fix_one`, `promote_one`, `backup_upload` after rename.
+**Fix:** Update docs; no behavior change.
+---
+## Priority guide
+| Offense | Typical priority |
+|---------|------------------|
+| O3, O4 | P0 |
+| O1, O2, O5, O6, O7, O8 | P1 |
+| O9, monolith size | P2 |
+| O10 | P3 |

package/.agents/skills/skill-builder/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills/skill-builder/references/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/{skills → .agents/skills}/skill-builder/references/deterministic-elements.md RENAMED Viewed

@@ -1,6 +1,6 @@
 # Deterministic skill elements
-Source: [Anthropic — Skill authoring best practices](https://platform.claude.com/docs/en/agents-and-tools/agent-skills/best-practices) (scripts, progressive disclosure), [Lessons from Building Claude Code](thariq-x-post-skills.json), repo skill conventions (`skills/CLAUDE.md`), `docs/CODE_RULES.md`, host task tools (`TaskCreate`, `TodoWrite`, and equivalents).
+Source: [Anthropic — Skill authoring best practices](https://platform.claude.com/docs/en/agents-and-tools/agent-skills/best-practices) (scripts, progressive disclosure), [Lessons from Building Claude Code](thariq-x-post-skills.json), repo skill conventions (`skills/.claude/CLAUDE.md`), `docs/CODE_RULES.md`, host task tools (`TaskCreate`, `TodoWrite`, and equivalents).
 ## Contents

package/.agents/skills/skill-builder/templates/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills/skill-builder/workflows/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills/source-command-logifix/SKILL.md ADDED Viewed

@@ -0,0 +1,29 @@
+---
+name: "source-command-logifix"
+description: "Restore the Logitech Gaming Software (LCore) tray icon when it disappears on Windows"
+---
+# source-command-logifix
+Use this skill when the user asks to run the migrated source command `logifix`.
+## Command Template
+Run the logifix recovery script to restore the LCore tray icon. The procedure is verified in `sessions/System Support/2. Logitech Tray Icon Fix Recurrence.md` (2026-04-25).
+Execute the script via PowerShell 7+:
+```
+pwsh -NoProfile -File "$HOME\.Codex\skills\logifix\scripts\logifix.ps1"
+```
+The script will prompt for UAC once. Approve it. After it finishes, ask the user to confirm the tray icon is now visible.
+If the script reports that UAC was canceled, or LCore did not respond after both relaunch attempts, instruct the user:
+1. Press **Ctrl+Shift+Esc** to open Task Manager.
+2. Find **Windows Explorer** in the process list.
+3. Right-click → **Restart**.
+4. Re-run `/logifix`.
+The Task Manager Explorer restart is the guaranteed-correct fallback because it always hits the user's interactive session and the right elevation context, regardless of how the calling shell was launched.

package/.agents/skills/source-command-sr-loop/SKILL.md ADDED Viewed

@@ -0,0 +1,58 @@
+---
+name: "source-command-sr-loop"
+description: "Converging cleanup loop: repeat /simplify until clean, then repeat /code-review --fix until clean"
+---
+# source-command-sr-loop
+Use this skill when the user asks to run the migrated source command `sr-loop`.
+## Command Template
+Run the converging cleanup loop on the target the user names (a PR URL,
+a branch, or blank for the current branch's diff). Each phase invokes an
+existing skill and repeats it until a pass returns zero new findings.
+### Phase A — loop simplify
+1. Invoke the `simplify` skill (or its local equivalent, e.g. `e-simplify`)
+   on the target. Let it run its 4-lens review fan-out (reuse,
+   simplification, efficiency, altitude) and apply its fixes.
+2. After each pass: run the scoped test suite (test files beside the touched
+   code, not the full repo suite), commit once (commit body via
+   `git commit -F <file>`, 10-minute timeout — the pre-commit gate runs its
+   own tests), and push to the PR head branch. The PR stays draft.
+3. Repeat the invocation. Skips are sticky: carry every adjudicated skip
+   forward into the next pass as "already adjudicated — do not re-report: ..."
+   context, or the loop never converges.
+4. Phase A converges when a full pass returns zero new findings. When the
+   previous pass changed only a line or two, a single combined-lens
+   confirmation agent may serve as the final pass.
+### Phase B — loop code-review with fixes
+1. Run the real code-review command through a headless subprocess in the
+   target worktree: `claude -p "/code-review low --fix" --max-turns 40` with a
+   10-minute timeout, reading its final text for findings and applied fixes.
+   (The skill's `disable-model-invocation` flag blocks only programmatic tool
+   starts; a `-p` prompt arrives as user input and loads normally.) Fallback
+   when the headless run fails: invoke the `e-code-review` skill with
+   `low --fix` — same review-and-fix contract, no invocation restriction.
+2. After any pass that changed files: test, commit, push as in Phase A.
+3. Repeat until a pass reports zero findings. Phase B usually converges in
+   one pass when Phase A ran first.
+### Finish
+Report: passes run per phase, commits pushed with hashes, fixes applied, and
+the standing skip list with reasons.
+### Constraints (observed under headless runs)
+- Run this loop in the main conversation. A wrapper subagent cannot see the
+  review skills, and a nested agent's own subagent reports route to the main
+  conversation instead of back to it.
+- Scope test runs to the touched files; a full-suite run can outlive one
+  foreground tool call.
+- A hung or newly slow test suite is a finding to investigate, not an
+  inconvenience.

package/.agents/skills/task-build/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills/update/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills-archived/AGENTS.md ADDED Viewed

@@ -0,0 +1,45 @@
+# skills-archived
+Versioned archive of retired Claude Code skills. These trees stay in the repository for reference and history but are not shipped to users.
+## Layout
+Each subdirectory is one retired skill with its historical `SKILL.md`, reference docs, scripts, and tests. Live runtime helpers for convergence and Codex review live under `packages/claude-dev-env/_shared/pr-loop/scripts/` — not in these trees.
+## Installer behavior
+`bin/install.mjs` copies skills only from `.agents/skills/`. This directory is outside that path and outside the `.claude/skills` lookup pointer, so archived skills never install to `~/.agents/skills/`.
+`bin/ever-shipped-skills.mjs` still lists retired skill names so a full install prunes stale copies from prior releases.
+## Archived skills
+| Skill | Role |
+|---|---|
+| `anthropic-plan` | Source-grounded plan packet workflow |
+| `auditing-claude-config` | Claude Code startup instruction audit |
+| `beat-sheet` | Single-line beat formatting |
+| `bugteam` | Open PR audit-fix loop |
+| `closeout` | Session obstacle harvest to GitHub issues |
+| `codex-review` | Local Codex PR reviewer (runtime scripts in `_shared`) |
+| `comments` | PR review comment guide |
+| `condensing-instructions` | Instruction slimming for Claude 5 |
+| `copilot-finding-triage` | Copilot gate finding tiering |
+| `copilot-review` | Copilot reviewer babysitter |
+| `descriptions` | PR description guide |
+| `emergencies` | Emergency change guide |
+| `grokify` | Grok Build handoff prompt |
+| `plan-to-pr` | Plan packet to PR workflow |
+| `pr-converge` | Paced PR convergence loop (runtime scripts in `_shared`) |
+| `pr-fix-protocol` | Reviewer finding fix protocol |
+| `pr-loop-cloud-transport` | Cloud session gh-to-MCP transport |
+| `pr-loop-lifecycle` | PR-loop open and close |
+| `recall` | Obsidian vault recall |
+| `release-notes-html` | Session release-notes HTML page |
+| `remember` | Obsidian vault remember |
+| `reviewer-gates` | External reviewer availability gates |
+| `reviews` | Code review guide |
+| `show` | Inline visual explanations |
+| `split-pr` | PR split analyzer |
+Prompt-generator dependency skills (`agent-prompt`, `pmin`, `pmax`, `pmid`, `prompt-generator`) shipped via the `@jl-cmd/prompt-generator` npm package and never had source trees in this repository.

package/.agents/skills-archived/anthropic-plan/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/{skills → .agents/skills-archived}/anthropic-plan/AGENTS.md RENAMED Viewed

@@ -19,7 +19,7 @@ Creates a repo-local plan packet under `docs/plans/<slug>/` by running the `plan
 | `SKILL.md` | Entry point. Full planning protocol, workflow contract, packet shape, and validation rules. |
 | `workflow/plan-packet.mjs` | Main workflow script. Reads repo context, writes the packet, runs the validator, spawns `plan-packet-validator`, runs the reuse audit, builds the visual HTML, and returns the packet path. |
 | `workflow/plan-packet.contract.test.mjs` | Contract tests for the workflow script. |
-| `packages/claude-dev-env/skills/anthropic-plan/scripts/validate_packet.py` | Deterministic validator: checks required files, open questions, source-map strength, TDD coverage, and `packet.json` consistency. Exits with code 2 on failure. |
+| `packages/claude-dev-env/.agents/skills-archived/anthropic-plan/scripts/validate_packet.py` | Deterministic validator: checks required files, open questions, source-map strength, TDD coverage, and `packet.json` consistency. Exits with code 2 on failure. |
 | `templates/visual-plan.template.html` | Template for the single-file offline visual HTML the workflow builds after validation. |
 ## Entry point

package/.agents/skills-archived/anthropic-plan/scripts/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills-archived/anthropic-plan/scripts/anthropic_plan_scripts_constants/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills-archived/anthropic-plan/templates/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/{skills → .agents/skills-archived}/anthropic-plan/test_skill_contract.py RENAMED Viewed

@@ -6,11 +6,11 @@ from pathlib import Path
 SKILL_DIRECTORY = Path(__file__).resolve().parent
-CLAUDE_DIRECTORY = SKILL_DIRECTORY.parent.parent
+PACKAGE_ROOT = SKILL_DIRECTORY.parent.parent.parent
 SKILL_PATH = SKILL_DIRECTORY / "SKILL.md"
-PLAN_COMMAND_PATH = CLAUDE_DIRECTORY / "commands" / "plan.md"
-SR_LOOP_COMMAND_PATH = CLAUDE_DIRECTORY / "commands" / "sr-loop.md"
-VALIDATOR_AGENT_PATH = CLAUDE_DIRECTORY / "agents" / "plan-packet-validator.md"
+PLAN_COMMAND_PATH = PACKAGE_ROOT / "commands" / "plan.md"
+SR_LOOP_COMMAND_PATH = PACKAGE_ROOT / "commands" / "sr-loop.md"
+VALIDATOR_AGENT_PATH = PACKAGE_ROOT / ".agents" / "agents" / "plan-packet-validator.md"
 def test_skill_invokes_plan_packet_workflow() -> None:

package/.agents/skills-archived/anthropic-plan/workflow/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills-archived/auditing-claude-config/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills-archived/bugteam/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills-archived/bugteam/reference/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills-archived/bugteam/reference/obstacles/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills-archived/bugteam/scripts/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/{skills → .agents/skills-archived}/bugteam/scripts/bugteam_code_rules_gate.py RENAMED Viewed

@@ -9,8 +9,18 @@ from __future__ import annotations
 import sys
 from pathlib import Path
+_here = Path(__file__).resolve()
+_skills_home_directory = _here.parents[3]
+_package_or_user_home = (
+    _skills_home_directory.parent
+    if _skills_home_directory.name == ".agents"
+    else _skills_home_directory
+)
+_shared_under_package = _package_or_user_home / "_shared" / "pr-loop" / "scripts"
 _shared_pr_loop_scripts_directory = (
-    Path(__file__).resolve().parents[3] / "_shared" / "pr-loop" / "scripts"
+    _shared_under_package
+    if _shared_under_package.is_dir()
+    else _package_or_user_home / ".claude" / "_shared" / "pr-loop" / "scripts"
 )
 if str(_shared_pr_loop_scripts_directory) not in sys.path:
     sys.path.insert(0, str(_shared_pr_loop_scripts_directory))

package/{skills → .agents/skills-archived}/bugteam/scripts/bugteam_fix_hookspath.py RENAMED Viewed

@@ -9,8 +9,18 @@ from __future__ import annotations
 import sys
 from pathlib import Path
+_here = Path(__file__).resolve()
+_skills_home_directory = _here.parents[3]
+_package_or_user_home = (
+    _skills_home_directory.parent
+    if _skills_home_directory.name == ".agents"
+    else _skills_home_directory
+)
+_shared_under_package = _package_or_user_home / "_shared" / "pr-loop" / "scripts"
 _shared_pr_loop_scripts_directory = (
-    Path(__file__).resolve().parents[3] / "_shared" / "pr-loop" / "scripts"
+    _shared_under_package
+    if _shared_under_package.is_dir()
+    else _package_or_user_home / ".claude" / "_shared" / "pr-loop" / "scripts"
 )
 if str(_shared_pr_loop_scripts_directory) not in sys.path:
     sys.path.insert(0, str(_shared_pr_loop_scripts_directory))

package/{skills → .agents/skills-archived}/bugteam/scripts/bugteam_preflight.py RENAMED Viewed

@@ -9,8 +9,18 @@ from __future__ import annotations
 import sys
 from pathlib import Path
+_here = Path(__file__).resolve()
+_skills_home_directory = _here.parents[3]
+_package_or_user_home = (
+    _skills_home_directory.parent
+    if _skills_home_directory.name == ".agents"
+    else _skills_home_directory
+)
+_shared_under_package = _package_or_user_home / "_shared" / "pr-loop" / "scripts"
 _shared_pr_loop_scripts_directory = (
-    Path(__file__).resolve().parents[3] / "_shared" / "pr-loop" / "scripts"
+    _shared_under_package
+    if _shared_under_package.is_dir()
+    else _package_or_user_home / ".claude" / "_shared" / "pr-loop" / "scripts"
 )
 if str(_shared_pr_loop_scripts_directory) not in sys.path:
     sys.path.insert(0, str(_shared_pr_loop_scripts_directory))

package/.agents/skills-archived/bugteam/scripts/bugteam_scripts_constants/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/{skills → .agents/skills-archived}/bugteam/scripts/reflow_skill_md.py RENAMED Viewed

@@ -1,16 +1,16 @@
-"""Reflow packages/claude-dev-env/skills/bugteam/SKILL.md to 80 columns.
+"""Reflow packages/claude-dev-env/.agents/skills-archived/bugteam/SKILL.md to 80 columns.
 Merge soft line breaks outside fenced blocks (space join; URL path fragments
 joined without a space only inside unfinished markdown link targets), then
 wrap with textwrap. Preserves fenced blocks verbatim.
-Same algorithm as ``packages/claude-dev-env/skills/pr-converge/scripts/reflow_skill_md.py``
+Same algorithm as ``packages/claude-dev-env/.agents/skills-archived/pr-converge/scripts/reflow_skill_md.py``
 from https://github.com/jl-cmd/claude-dev-env/pull/349 (branch
 ``cursor/pr-converge-skill-line-wrap-ecd1``); ``SKILL_PATH`` points at bugteam
 ``SKILL.md``. Link reference definitions (``[id]: url``) are treated as logical
 line starts so they are not merged with prior paragraphs.
-Run: python3 packages/claude-dev-env/skills/bugteam/scripts/reflow_skill_md.py
+Run: python3 packages/claude-dev-env/.agents/skills-archived/bugteam/scripts/reflow_skill_md.py
 """
 from __future__ import annotations

package/.agents/skills-archived/codex-review/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills-archived/codex-review/reference/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/.agents/skills-archived/codex-review/scripts/codex_review_scripts_constants/.claude/CLAUDE.md ADDED Viewed

	@@ -0,0 +1 @@
1	+ @../AGENTS.md

package/{skills → .agents/skills-archived}/codex-review/scripts/codex_usage_probe.py RENAMED Viewed

@@ -43,8 +43,18 @@ from datetime import datetime, timezone
 from pathlib import Path
 from typing import IO, Protocol
+_here = Path(__file__).resolve()
+_skills_home_directory = _here.parents[3]
+_package_or_user_home = (
+    _skills_home_directory.parent
+    if _skills_home_directory.name == ".agents"
+    else _skills_home_directory
+)
+_shared_under_package = _package_or_user_home / "_shared" / "process-tree" / "scripts"
 _shared_process_tree_scripts_directory = (
-    Path(__file__).resolve().parents[3] / "_shared" / "process-tree" / "scripts"
+    _shared_under_package
+    if _shared_under_package.is_dir()
+    else _package_or_user_home / ".claude" / "_shared" / "process-tree" / "scripts"
 )
 if str(_shared_process_tree_scripts_directory) not in sys.path:
     sys.path.insert(0, str(_shared_process_tree_scripts_directory))