diff --git a/.claude/skills/piker-profiling/SKILL.md b/.agents/skills/piker-profiling/SKILL.md similarity index 100% rename from .claude/skills/piker-profiling/SKILL.md rename to .agents/skills/piker-profiling/SKILL.md diff --git a/.claude/skills/piker-profiling/patterns.md b/.agents/skills/piker-profiling/patterns.md similarity index 100% rename from .claude/skills/piker-profiling/patterns.md rename to .agents/skills/piker-profiling/patterns.md diff --git a/.claude/skills/piker-slang/SKILL.md b/.agents/skills/piker-slang/SKILL.md similarity index 100% rename from .claude/skills/piker-slang/SKILL.md rename to .agents/skills/piker-slang/SKILL.md diff --git a/.claude/skills/piker-slang/dictionary.md b/.agents/skills/piker-slang/dictionary.md similarity index 100% rename from .claude/skills/piker-slang/dictionary.md rename to .agents/skills/piker-slang/dictionary.md diff --git a/.claude/skills/piker-slang/examples.md b/.agents/skills/piker-slang/examples.md similarity index 100% rename from .claude/skills/piker-slang/examples.md rename to .agents/skills/piker-slang/examples.md diff --git a/.claude/skills/pyqtgraph-optimization/SKILL.md b/.agents/skills/pyqtgraph-optimization/SKILL.md similarity index 100% rename from .claude/skills/pyqtgraph-optimization/SKILL.md rename to .agents/skills/pyqtgraph-optimization/SKILL.md diff --git a/.claude/skills/pyqtgraph-optimization/examples.md b/.agents/skills/pyqtgraph-optimization/examples.md similarity index 100% rename from .claude/skills/pyqtgraph-optimization/examples.md rename to .agents/skills/pyqtgraph-optimization/examples.md diff --git a/.claude/skills/timeseries-optimization/SKILL.md b/.agents/skills/timeseries-optimization/SKILL.md similarity index 100% rename from .claude/skills/timeseries-optimization/SKILL.md rename to .agents/skills/timeseries-optimization/SKILL.md diff --git a/.claude/skills/timeseries-optimization/numpy-patterns.md b/.agents/skills/timeseries-optimization/numpy-patterns.md similarity index 100% rename from .claude/skills/timeseries-optimization/numpy-patterns.md rename to .agents/skills/timeseries-optimization/numpy-patterns.md diff --git a/.claude/skills/timeseries-optimization/polars-patterns.md b/.agents/skills/timeseries-optimization/polars-patterns.md similarity index 100% rename from .claude/skills/timeseries-optimization/polars-patterns.md rename to .agents/skills/timeseries-optimization/polars-patterns.md diff --git a/.claude/skills/commit-msg/style-guide-reference.md b/.ai/commit-msg/style-guide-reference.md similarity index 93% rename from .claude/skills/commit-msg/style-guide-reference.md rename to .ai/commit-msg/style-guide-reference.md index 1e3ba061..0698cc62 100644 --- a/.claude/skills/commit-msg/style-guide-reference.md +++ b/.ai/commit-msg/style-guide-reference.md @@ -150,21 +150,18 @@ Common in piker commits (33.0% use colons): - File:line references not used (0 occurrences) - No WIP commits in analyzed set -### Claude-code Footer -When the written **patch** was assisted by claude-code, -include: +### Coding-harness Footer +When a coding harness assisted with the written patch, +identify the active harness, model, and provider: ``` -(this patch was generated in some part by [`claude-code`][claude-code-gh]) -[claude-code-gh]: https://github.com/anthropics/claude-code +(this patch was generated in some part by `` using `` (``)) ``` -When only the **commit msg** was written by claude-code -(human wrote the patch), use: +When it generated only the commit message, use: ``` -(this commit msg was generated in some part by [`claude-code`][claude-code-gh]) -[claude-code-gh]: https://github.com/anthropics/claude-code +(this commit msg was generated in some part by `` using `` (``)) ``` ## Piker-Specific Terms diff --git a/.ai/run-tests/test-harness-reference.md b/.ai/run-tests/test-harness-reference.md new file mode 100644 index 00000000..dc419d64 --- /dev/null +++ b/.ai/run-tests/test-harness-reference.md @@ -0,0 +1,291 @@ +# Test Harness Reference: piker + +This repository-local file supplements the canonical `/run-tests` skill. +Keep shared execution, worktree, failure-inspection, and cleanup policy in the +canonical `SKILL.md`; keep Piker commands, paths, fixtures, and known outcomes +here. + +## Project And Environment + +- Project/import: `piker` +- Test root: `tests/` +- Supported Python: `>=3.12,<3.14` +- Preferred complete environment: worktree-local `py313` inside the current + `nix develop` shell +- The flake shell pins CPython 3.13 and sets + `UV_PROJECT_ENVIRONMENT=py313`. +- Verify the interpreter, package resolution, and dependency import before + running tests. A bare `py313` may lack the Qt binding supplied by Nix. + +Use an already-provisioned `py313` only when `import piker` succeeds: + +```text +py313/bin/python -m pytest -p no:xonsh +``` + +If direct environment paths are unavailable, an existing uv environment can +be used without changing it, subject to the same import check: + +```text +UV_PROJECT_ENVIRONMENT=py313 uv run --frozen --no-sync python -m pytest -p no:xonsh +``` + +Ask before running provisioning commands such as: + +```text +UV_PROJECT_ENVIRONMENT=py313 uv sync --dev --all-extras --no-group lint +nix develop +nix-shell default.nix +``` + +`nix develop` is the current Wayland/Qt 6 shell. `default.nix` is the current +X11 shell. Do not use `develop.nix` for current testing; it retains the old +Python 3.11, Poetry, and Qt 5 stack. + +The current root-checkout `py313` resolves `piker` locally but fails +`import piker` outside Nix because PyQtGraph cannot import PyQt or PySide. Do +not treat that environment as test-ready and do not enter `nix develop` +without approval: its shell hook may recreate and sync `py313`. + +Plain `uv sync` does not include the testing group. The `dbs` dependency group +is also absent from normal dev-shell provisioning. + +## Commands + +Base command in the preferred environment: + +```text +py313/bin/python -m pytest -p no:xonsh +``` + +The explicit `-p no:xonsh` is required. The tracked comments-only +`pytest.ini` takes precedence over `pyproject.toml`, so the intended +`addopts = "-p no:xonsh"` and `testpaths = ["tests"]` are inactive. +Always pass a test path or node ID explicitly. + +Package-resolution check that does not import Piker's dependencies: + +```text +py313/bin/python -c 'import importlib.util, pathlib, sys; root = pathlib.Path.cwd().resolve(); spec = importlib.util.find_spec("piker"); mod = pathlib.Path(spec.origin).resolve(); print(sys.executable); print(mod); assert mod.is_relative_to(root)' +``` + +Dependency import check, required before collection or execution: + +```text +py313/bin/python -c 'import pathlib, piker, sys; root = pathlib.Path.cwd().resolve(); mod = pathlib.Path(piker.__file__).resolve(); print(sys.executable); print(mod); assert mod.is_relative_to(root)' +``` + +Safe core collection check: + +```text +py313/bin/python -m pytest -p no:xonsh -q --collect-only tests/test_watchlists.py tests/test_accounting.py tests/test_services.py tests/test_ems.py tests/test_feeds.py tests/test_cli.py +``` + +Default first-pass flags are `-q -x --tb=short --no-header` unless the user +requests otherwise. For actor-heavy tests, use one file or node per process +with an outer timeout: + +```text +timeout -k 5 300 py313/bin/python -m pytest -p no:xonsh -q +``` + +If an actor-heavy command exits `124` or `143`, retry that exact command once +and report both attempts. Never convert a retry pass into an unconditional +pass, and do not retry assertion, import, collection, or configuration +failures. + +## Scope And Path Resolution + +Resolve bare test filenames beneath `tests/`. Preserve complete node IDs and +apply `-k` only within explicitly selected paths. There is no marker-based +offline/full-suite split, so never use `pytest tests` as a deterministic +default. + +Deterministic or local first-pass targets: + +- `tests/test_watchlists.py` +- `tests/test_storage_audit.py` +- `tests/test_store_cli.py` +- `tests/test_backfill_audit_snippet.py` +- `tests/test_ib_history.py` +- `tests/test_ib_method_proxy.py` +- `tests/test_history_backfill.py` +- `tests/test_ldshm.py` +- `tests/test_accounting.py::test_account_file_default_empty` +- `tests/test_services.py::test_runtime_boot` +- `tests/test_services.py::test_datad_spawn` +- `tests/test_ems.py::test_ems_err_on_bad_broker` + +Require explicit authorization before running: + +- `tests/test_feeds.py` - live Binance/Kraken feeds; +- `tests/test_services.py::test_ensure_datafeed_actors` - live Kraken feed; +- `tests/test_services.py::test_ensure_ems_in_paper_actors` - paper EMS with + live Kraken symbology/feed access; +- `tests/test_ems.py::test_multi_fill_positions` - live backend setup and + persisted state; +- `tests/test_accounting.py::test_paper_ledger_position_calcs` - tracked + fixtures plus possible live symcache generation; +- `tests/test_accounting.py::test_ib_account_with_duplicated_mktids` - active + broker/account configuration and state writes; +- `tests/test_dpi_font.py` - Qt/UI import and user-config side effects; +- `tests/test_docker_services.py` - optional dependencies and containers; +- `tests/test_questrade.py` - obsolete credentialed imports. + +`tests/test_cli.py` is currently hard-skipped. It is not an active CLI +regression gate. + +Never execute `piker store anal` or `piker store shm --write-parquet` +as tests. They are mutating or interactive operational commands. + +## Project-Specific Flags And Backend Matrix + +| Flag | Purpose | +|---|---| +| `--ll LEVEL` | Piker log level | +| `--confdir PATH` | Override `piker.config._config_dir` | +| `--spawn-backend trio|mp_spawn|mp_forkserver` | Tractor process backend | +| `--tpt-proto PROTO` | Tractor transport; one protocol per session | +| `--tpdb` / `--debug-mode` | Tractor plugin debug mode | +| `--pdb` | Standard pytest debugger | +| `-s` | No capture; required with `--pdb` and `open_test_pikerd` | + +Do not invent `network`, `offline`, `docker`, `gui`, or `broker` markers; none +currently exists. `CI=1` is not an offline selector: feed tests still leave a +live Kraken case enabled. Current Piker tests exercise TCP; do not assume the +whole suite supports UDS merely because the Tractor plugin exposes it. + +## Fixture Invariants + +- The session `confdir` fixture does nothing unless `--confdir` is passed. + Its claimed `tests/data` fallback is not implemented and that directory is + absent. +- Function-scoped `tmpconfdir` changes process-global config state and does + not restore the previous path. Use separate pytest processes when + diagnosing state leakage. +- `open_test_pikerd` passes the temporary config path to child actors through + `tractor_runtime_overrides`. +- `tests/_inputs/trades_binance_paper.toml` and + `tests/_inputs/account.binance.paper.toml` are used in place. Accounting + contexts can write them on exit. Inspect `git diff -- tests/_inputs` after + any selected accounting case. +- Importing `tests/test_dpi_font.py` constructs module-global font objects + before fixtures can isolate config. If explicitly requested, isolate + `XDG_CONFIG_HOME` before Python starts and use the proper Qt/Nix shell. +- Piker and the installed Tractor pytest plugin do not provide a + repository-local process or socket reaper. Never apply historical broad + `pkill -f tractor._child` guidance automatically. +- The function-scoped autouse `shm_leak_tracker` fixture wraps Tractor's + current-process `SharedMemory` factory. It tracks only successful + `create=True` calls, restores the pre-test `_known_tokens` cache, and + unlinks exact surviving names before failing the leaking test. It never + scans `/dev/shm` or unlinks attachments created by another process. + +## Test Layout + +```text +tests/ + conftest.py Piker options, config fixtures, Tractor plugin + _inputs/ tracked ledger/account fixtures + test_accounting.py config, ledgers, accounts, and position math + test_cli.py legacy CLI suite; hard-skipped + test_docker_services.py container integrations; optional deps, skipped + test_dpi_font.py Qt DPI/font behavior + test_ems.py actor, EMS, and paper-position behavior + test_feeds.py live Binance/Kraken feeds and shared memory + test_ib_history.py deterministic IB history request formatting + test_ib_method_proxy.py deterministic IB asyncio proxy routing + test_history_backfill.py deterministic history/SHM orchestration + test_ldshm.py SHM unpublished-slot guard + test_questrade.py obsolete credentialed tests; skipped + test_services.py pikerd/datad/feed/EMS actor lifecycle + test_store_cli.py storage command help and diagnostics UX + test_storage_audit.py read-only NativeDB audit and JSON CLI + test_backfill_audit_snippet.py + disposable xonsh qualification helpers + test_watchlists.py deterministic watchlist JSON operations +``` + +## Change-To-Test Mapping + +| Changed area | Run first | Caveat | +|---|---|---| +| `piker/watchlists/` | `tests/test_watchlists.py` | CLI suite is skipped | +| `piker/storage/_audit.py`, `piker/storage/cli.py` | `tests/test_storage_audit.py` | direct Typer app, no actor | +| `piker/storage/cli.py` command UX | `tests/test_store_cli.py` | fake SHM/runtime, no mutation | +| `snippets/nativedb_backfill_audit.xsh` | `tests/test_backfill_audit_snippet.py` | disposable paths only | +| `piker/brokers/ib/api.py`, `feed.py` history | `tests/test_ib_history.py` | fake client, no network | +| `piker/brokers/ib/api.py` method proxy | `tests/test_ib_method_proxy.py` | fake channel, no network | +| `piker/tsp/_history.py` | `tests/test_history_backfill.py` | fake provider/storage/SHM | +| `piker/storage/cli.py` SHM null-slot guard | `tests/test_ldshm.py` | synthetic timestamps, no SHM mutation | +| `piker/config.py` | `test_account_file_default_empty` | root-network test has a known mismatch | +| `piker/accounting/` | targeted accounting node | some cases use live/configured state | +| `piker/ui/_style.py`, `piker/ui/qt.py` | `tests/test_dpi_font.py` | GUI/config-isolated opt-in | +| `piker/service/_actor_runtime.py`, `_registry.py`, `_mngr.py` | `test_runtime_boot` | then `test_datad_spawn` | +| `piker/service/`, `piker/data/_daemon.py` | `test_datad_spawn` | feed lifecycle cases are live | +| `piker/data/feed.py`, `flows.py`, `_sharedmem.py`, `_sampling.py` | collect first | feed execution needs live permission | +| `piker/clearing/` | `test_ems_err_on_bad_broker` | multi-fill case is live/persisted | +| `piker/brokers/binance/`, `kraken/` | selected feed/accounting node | live network | +| `piker/brokers/ib/` | duplicated-market-ID node | controlled account config required | +| Docker/service adapters | `tests/test_docker_services.py` | optional deps and containers | +| project, lock, or Nix files | import check and safe collection | full collection is not safe by default | + +Prefer deterministic filesystem/config tests, then local actor-runtime nodes, +then explicitly approved live broker, GUI, or container coverage. + +## Quick Checks + +```text +py313/bin/python -c 'import importlib.util, pathlib, sys; root = pathlib.Path.cwd().resolve(); spec = importlib.util.find_spec("piker"); mod = pathlib.Path(spec.origin).resolve(); print(sys.executable); print(mod); assert mod.is_relative_to(root)' +py313/bin/python -c 'import pathlib, piker, sys; root = pathlib.Path.cwd().resolve(); mod = pathlib.Path(piker.__file__).resolve(); print(sys.executable); print(mod); assert mod.is_relative_to(root)' +py313/bin/python -m pytest -p no:xonsh -q tests/test_watchlists.py +py313/bin/python -m pytest -p no:xonsh -q tests/test_accounting.py::test_account_file_default_empty +timeout -k 5 300 py313/bin/python -m pytest -p no:xonsh -q tests/test_services.py::test_runtime_boot +timeout -k 5 300 py313/bin/python -m pytest -p no:xonsh -q tests/test_services.py::test_datad_spawn +timeout -k 5 300 py313/bin/python -m pytest -p no:xonsh -q tests/test_ems.py::test_ems_err_on_bad_broker +``` + +## Known Outcomes + +- The current root-checkout `py313` fails `import piker` outside the Nix shell + with `ImportError: PyQtGraph requires one of PyQt5, PyQt6, PySide2 or + PySide6`. This is an incomplete environment, not an application regression. +- `tests/test_accounting.py::test_root_conf_networking_section` currently + expects `network.tsdb`, which is absent from the tracked config template. + Match the current `KeyError: 'tsdb'` before classifying it as the known + repository mismatch. +- `tests/test_docker_services.py` is marked skipped but imports + `elasticsearch` first. Without the `dbs` group it fails collection rather + than skipping. +- `tests/test_questrade.py` is marked skipped but imports undeclared `asks` + through the legacy broker module before marks apply. +- `test_open_orders_reloaded` and `test_dark_order_clearing` in + `tests/test_ems.py` contain only ellipsis bodies. A pass does not verify the + named behavior. +- Treat `.pytest_cache` feed IDs using old FQME forms as stale historical + cache, not current known failures. + +Do not classify every `TooSlowError`, timeout, or child survivor as the known +Tractor teardown wedge. Match the selected node and the documented +second-runtime/lingering-child signature. + +## Tractor Runtime Notes + +The suite loads `tractor._testing.pytest`. Defaults are the `trio` spawn +backend and TCP transport. Registry addresses are normally session-unique, +except `tests/test_services.py::test_runtime_boot`, which binds +`127.0.0.1:6666`. Check that fixed port only for that node; ordinary tests do +not require Piker's production `127.0.0.1:6116` registry address. + +Current repository concurrency notes document an intermittent second +in-process `pikerd` boot wedge with a lingering broker child and unread parent +IPC bytes. Use the outer timeout for actor-heavy nodes. Retry an exact command +once only after status `124` or `143`, and report both attempts. + +Ordinary actor tests support normal capture. Diagnose a capture-dependent hang +by retrying only the exact node with `-s`. Standard `--pdb` with +`open_test_pikerd` requires `-s`; Tractor's `--tpdb` is a separate option. + +After abnormal exit, inspect only descendants, sockets, and shared-memory +objects attributable to the exact pytest session. Ask before signaling or +unlinking anything. diff --git a/.ai/workflow-state.json b/.ai/workflow-state.json new file mode 100644 index 00000000..269876ed --- /dev/null +++ b/.ai/workflow-state.json @@ -0,0 +1 @@ +{"version": 1, "backend": "neutral"} diff --git a/.claude/settings.local.json b/.claude/settings.local.json deleted file mode 100644 index 5404a5dd..00000000 --- a/.claude/settings.local.json +++ /dev/null @@ -1,11 +0,0 @@ -{ - "permissions": { - "allow": [ - "Bash(chmod:*)", - "Bash(/tmp/piker_commits.txt)", - "Bash(python:*)" - ], - "deny": [], - "ask": [] - } -} diff --git a/.claude/skills/commit-msg/SKILL.md b/.claude/skills/commit-msg/SKILL.md deleted file mode 100644 index 0a9f69c4..00000000 --- a/.claude/skills/commit-msg/SKILL.md +++ /dev/null @@ -1,84 +0,0 @@ ---- -name: commit-msg -description: > - Generate piker-style git commit messages from - staged changes or prompt input, following the - style guide learned from 500 repo commits. -argument-hint: "[optional-scope-or-description]" -disable-model-invocation: true -allowed-tools: Bash(git *), Read, Grep, Glob, Write ---- - -## Current staged changes -!`git diff --staged --stat` - -## Recent commit style reference -!`git log --oneline -10` - -# Piker Git Commit Message Generator - -Generate a commit message from the staged diff above -following the piker project's conventions (learned from -analyzing 500 repo commits). - -If `$ARGUMENTS` is provided, use it as scope or -description context for the commit message. - -For the full style guide with verb frequencies, -section markers, abbreviations, piker-specific terms, -and examples, see -[style-guide-reference.md](./style-guide-reference.md). - -## Quick Reference - -- **Subject**: ~50 chars, present tense verb, use - backticks for code refs -- **Body**: only for complex/multi-file changes, - 67 char line max -- **Section markers**: Also, / Deats, / Other, -- **Bullets**: use `-` style -- **Tone**: technical but casual (piker style) - -## Claude-code Footer - -When the written **patch** was assisted by -claude-code, include: - -``` -(this patch was generated in some part by [`claude-code`][claude-code-gh]) -[claude-code-gh]: https://github.com/anthropics/claude-code -``` - -When only the **commit msg** was written by -claude-code (human wrote the patch), use: -``` -(this commit msg was generated in some part by [`claude-code`][claude-code-gh]) -[claude-code-gh]: https://github.com/anthropics/claude-code -``` - -## Output Instructions - -When generating a commit message: - -1. Analyze the staged diff (injected above via - dynamic context) to understand all changes. -2. If `$ARGUMENTS` provides a scope (e.g., - `.ib.feed`) or description, incorporate it into - the subject line. -3. Write the subject line following verb + backtick - conventions from the - [style guide](./style-guide-reference.md). -4. Add body only for multi-file or complex changes. -5. Write the message to a file in the repo's - `.claude/` subdir with filename format: - `__commit_msg.md` - where `` is from `date --iso-8601=seconds`. - Also write a copy to - `.claude/git_commit_msg_LATEST.md` - (overwrite if exists). - ---- - -**Analysis date:** 2026-01-27 -**Commits analyzed:** 500 from piker repository -**Maintained by:** Tyler Goodlet diff --git a/.claude/skills/piker-profiling b/.claude/skills/piker-profiling new file mode 120000 index 00000000..2b3636a0 --- /dev/null +++ b/.claude/skills/piker-profiling @@ -0,0 +1 @@ +../../.agents/skills/piker-profiling \ No newline at end of file diff --git a/.claude/skills/piker-slang b/.claude/skills/piker-slang new file mode 120000 index 00000000..85f9cbec --- /dev/null +++ b/.claude/skills/piker-slang @@ -0,0 +1 @@ +../../.agents/skills/piker-slang \ No newline at end of file diff --git a/.claude/skills/pyqtgraph-optimization b/.claude/skills/pyqtgraph-optimization new file mode 120000 index 00000000..b1ca4594 --- /dev/null +++ b/.claude/skills/pyqtgraph-optimization @@ -0,0 +1 @@ +../../.agents/skills/pyqtgraph-optimization \ No newline at end of file diff --git a/.claude/skills/timeseries-optimization b/.claude/skills/timeseries-optimization new file mode 120000 index 00000000..2bd3abd1 --- /dev/null +++ b/.claude/skills/timeseries-optimization @@ -0,0 +1 @@ +../../.agents/skills/timeseries-optimization \ No newline at end of file diff --git a/.gitignore b/.gitignore index 0a058bba..583063c3 100644 --- a/.gitignore +++ b/.gitignore @@ -107,6 +107,9 @@ ENV/ .git/ # any commit-msg gen tmp files +.claude/skills/commit-msg/msgs/ +.claude/skills/commit-msg/conf.toml +.claude/git_commit_msg_LATEST.md .claude/*_commit_*.md .claude/*_commit*.toml @@ -130,3 +133,362 @@ gitea/ # LLM conversations that should remain private docs/conversations/ + +# BEGIN ai.skillz: direct:symlink:claude:gish +/.claude/skills/gish +# END ai.skillz: direct:symlink:claude:gish + +# BEGIN ai.skillz: direct:symlink:opencode:gish +/.opencode/skills/gish +# END ai.skillz: direct:symlink:opencode:gish + +# BEGIN ai.skillz: runtime:open-wkt +/wkts/ +# END ai.skillz: runtime:open-wkt + +# BEGIN ai.skillz: direct:symlink:claude:open-wkt +/.claude/skills/open-wkt +# END ai.skillz: direct:symlink:claude:open-wkt + +# BEGIN ai.skillz: direct:symlink:opencode:open-wkt +/.opencode/skills/open-wkt +# END ai.skillz: direct:symlink:opencode:open-wkt + +# BEGIN ai.skillz: direct:symlink:claude:run-tests +/.claude/skills/run-tests/SKILL.md +# END ai.skillz: direct:symlink:claude:run-tests + +# BEGIN ai.skillz: direct:symlink:opencode:run-tests +/.opencode/skills/run-tests/SKILL.md +# END ai.skillz: direct:symlink:opencode:run-tests + +# BEGIN ai.skillz: direct:symlink:claude:close-wkt +/.claude/skills/close-wkt +# END ai.skillz: direct:symlink:claude:close-wkt + +# BEGIN ai.skillz: direct:symlink:opencode:close-wkt +/.opencode/skills/close-wkt +# END ai.skillz: direct:symlink:opencode:close-wkt + +# BEGIN ai.skillz: runtime:code-review +.ai/code-review/reports/ +# END ai.skillz: runtime:code-review + +# BEGIN ai.skillz: direct:symlink:claude:code-review +/.claude/skills/code-review +# END ai.skillz: direct:symlink:claude:code-review + +# BEGIN ai.skillz: direct:symlink:opencode:code-review +/.opencode/skills/code-review +# END ai.skillz: direct:symlink:opencode:code-review + +# BEGIN ai.skillz: runtime:code-review-changes +.ai/state/review/ +.claude/review_context.md +.claude/review_regression.md +.claude/review_replies/ +# END ai.skillz: runtime:code-review-changes + +# BEGIN ai.skillz: direct:symlink:claude:code-review-changes +/.claude/skills/code-review-changes +# END ai.skillz: direct:symlink:claude:code-review-changes + +# BEGIN ai.skillz: direct:symlink:opencode:code-review-changes +/.opencode/skills/code-review-changes +# END ai.skillz: direct:symlink:opencode:code-review-changes + +# BEGIN ai.skillz: runtime:commit-msg +.ai/state/commit-msg/ +.ai/state/migrations/ +.ai/commit-msg/conf.toml +.claude/skills/commit-msg/msgs/ +.claude/git_commit_msg_LATEST.md +# END ai.skillz: runtime:commit-msg + +# BEGIN ai.skillz: direct:symlink:claude:commit-msg +/.claude/skills/commit-msg/SKILL.md +# END ai.skillz: direct:symlink:claude:commit-msg + +# BEGIN ai.skillz: direct:symlink:opencode:commit-msg +/.opencode/skills/commit-msg/SKILL.md +# END ai.skillz: direct:symlink:opencode:commit-msg + +# BEGIN ai.skillz: direct:symlink:claude:commit-plan +/.claude/skills/commit-plan +# END ai.skillz: direct:symlink:claude:commit-plan + +# BEGIN ai.skillz: direct:symlink:opencode:commit-plan +/.opencode/skills/commit-plan +# END ai.skillz: direct:symlink:opencode:commit-plan + +# BEGIN ai.skillz: direct:symlink:claude:dep-supersede-scan +/.claude/skills/dep-supersede-scan +# END ai.skillz: direct:symlink:claude:dep-supersede-scan + +# BEGIN ai.skillz: direct:symlink:opencode:dep-supersede-scan +/.opencode/skills/dep-supersede-scan +# END ai.skillz: direct:symlink:opencode:dep-supersede-scan + +# BEGIN ai.skillz: direct:symlink:claude:resolve-conflicts +/.claude/skills/resolve-conflicts +# END ai.skillz: direct:symlink:claude:resolve-conflicts + +# BEGIN ai.skillz: direct:symlink:opencode:resolve-conflicts +/.opencode/skills/resolve-conflicts +# END ai.skillz: direct:symlink:opencode:resolve-conflicts + +# BEGIN ai.skillz: direct:symlink:claude:git-mgmt +/.claude/skills/git-mgmt +# END ai.skillz: direct:symlink:claude:git-mgmt + +# BEGIN ai.skillz: direct:symlink:opencode:git-mgmt +/.opencode/skills/git-mgmt +# END ai.skillz: direct:symlink:opencode:git-mgmt + +# BEGIN ai.skillz: direct:symlink:claude:harness-perf +/.claude/skills/harness-perf +# END ai.skillz: direct:symlink:claude:harness-perf + +# BEGIN ai.skillz: direct:symlink:opencode:harness-perf +/.opencode/skills/harness-perf +# END ai.skillz: direct:symlink:opencode:harness-perf + +# BEGIN ai.skillz: direct:symlink:claude:inter-skill-review +/.claude/skills/inter-skill-review +# END ai.skillz: direct:symlink:claude:inter-skill-review + +# BEGIN ai.skillz: direct:symlink:opencode:inter-skill-review +/.opencode/skills/inter-skill-review +# END ai.skillz: direct:symlink:opencode:inter-skill-review + +# BEGIN ai.skillz: direct:symlink:claude:opencode-cleaning +/.claude/skills/opencode-cleaning +# END ai.skillz: direct:symlink:claude:opencode-cleaning + +# BEGIN ai.skillz: direct:symlink:opencode:opencode-cleaning +/.opencode/skills/opencode-cleaning +# END ai.skillz: direct:symlink:opencode:opencode-cleaning + +# BEGIN ai.skillz: direct:symlink:claude:plan-io +/.claude/skills/plan-io +# END ai.skillz: direct:symlink:claude:plan-io + +# BEGIN ai.skillz: direct:symlink:opencode:plan-io +/.opencode/skills/plan-io +# END ai.skillz: direct:symlink:opencode:plan-io + +# BEGIN ai.skillz: runtime:pr-msg +.ai/state/pr-msg/ +.ai/pr-msg/conf.toml +.claude/skills/pr-msg/msgs/ +.claude/skills/pr-msg/pr_msg_LATEST.md +# END ai.skillz: runtime:pr-msg + +# BEGIN ai.skillz: direct:symlink:claude:pr-msg +/.claude/skills/pr-msg/SKILL.md +/.claude/skills/pr-msg/references +/.claude/skills/pr-msg/scripts +# END ai.skillz: direct:symlink:claude:pr-msg + +# BEGIN ai.skillz: direct:symlink:opencode:pr-msg +/.opencode/skills/pr-msg/SKILL.md +/.opencode/skills/pr-msg/references +/.opencode/skills/pr-msg/scripts +# END ai.skillz: direct:symlink:opencode:pr-msg + +# BEGIN ai.skillz: direct:symlink:claude:prompt-io +/.claude/skills/prompt-io +# END ai.skillz: direct:symlink:claude:prompt-io + +# BEGIN ai.skillz: direct:symlink:opencode:prompt-io +/.opencode/skills/prompt-io +# END ai.skillz: direct:symlink:opencode:prompt-io + +# BEGIN ai.skillz: direct:symlink:claude:py-codestyle +/.claude/skills/py-codestyle +# END ai.skillz: direct:symlink:claude:py-codestyle + +# BEGIN ai.skillz: direct:symlink:opencode:py-codestyle +/.opencode/skills/py-codestyle +# END ai.skillz: direct:symlink:opencode:py-codestyle + +# BEGIN ai.skillz: runtime:taken-export +.ai/taken/exports/ +# END ai.skillz: runtime:taken-export + +# BEGIN ai.skillz: direct:symlink:claude:taken-export +/.claude/skills/taken-export +# END ai.skillz: direct:symlink:claude:taken-export + +# BEGIN ai.skillz: direct:symlink:opencode:taken-export +/.opencode/skills/taken-export +# END ai.skillz: direct:symlink:opencode:taken-export + +# BEGIN ai.skillz: direct:symlink:claude:yt-url-lookup +/.claude/skills/yt-url-lookup +# END ai.skillz: direct:symlink:claude:yt-url-lookup + +# BEGIN ai.skillz: direct:symlink:opencode:yt-url-lookup +/.opencode/skills/yt-url-lookup +# END ai.skillz: direct:symlink:opencode:yt-url-lookup + +# BEGIN ai.skillz: direct:symlink:opencode:command:run-tests +/.opencode/commands/run-tests.md +# END ai.skillz: direct:symlink:opencode:command:run-tests + +# BEGIN ai.skillz: direct:symlink:opencode:command:code-review +/.opencode/commands/code-review.md +# END ai.skillz: direct:symlink:opencode:command:code-review + +# BEGIN ai.skillz: direct:symlink:opencode:command:code-review-changes +/.opencode/commands/code-review-changes.md +# END ai.skillz: direct:symlink:opencode:command:code-review-changes + +# BEGIN ai.skillz: direct:symlink:opencode:command:commit-msg +/.opencode/commands/commit-msg.md +# END ai.skillz: direct:symlink:opencode:command:commit-msg + +# BEGIN ai.skillz: direct:symlink:opencode:command:commit-plan +/.opencode/commands/commit-plan.md +# END ai.skillz: direct:symlink:opencode:command:commit-plan + +# BEGIN ai.skillz: direct:symlink:opencode:command:opencode-cleaning +/.opencode/commands/opencode-cleaning.md +# END ai.skillz: direct:symlink:opencode:command:opencode-cleaning + +# BEGIN ai.skillz: direct:symlink:opencode:command:pr-msg +/.opencode/commands/pr-msg.md +# END ai.skillz: direct:symlink:opencode:command:pr-msg + +# BEGIN ai.skillz: direct:symlink:opencode:command:taken-export +/.opencode/commands/taken-export.md +# END ai.skillz: direct:symlink:opencode:command:taken-export + +# BEGIN ai.skillz: direct:symlink:claude:code-nav-refs +/.claude/skills/code-nav-refs +# END ai.skillz: direct:symlink:claude:code-nav-refs + +# BEGIN ai.skillz: direct:symlink:opencode:code-nav-refs +/.opencode/skills/code-nav-refs +# END ai.skillz: direct:symlink:opencode:code-nav-refs + +# BEGIN ai.skillz: direct:symlink:opencode:command:gish +/.opencode/commands/gish.md +# END ai.skillz: direct:symlink:opencode:command:gish + +# BEGIN ai.skillz: direct:symlink:opencode:command:resolve-conflicts +/.opencode/commands/resolve-conflicts.md +# END ai.skillz: direct:symlink:opencode:command:resolve-conflicts + +# BEGIN ai.skillz: direct:symlink:opencode:command:git-mgmt +/.opencode/commands/git-mgmt.md +# END ai.skillz: direct:symlink:opencode:command:git-mgmt + +# BEGIN ai.skillz: direct:symlink:opencode:command:open-wkt +/.opencode/commands/open-wkt.md +# END ai.skillz: direct:symlink:opencode:command:open-wkt + +# BEGIN ai.skillz: direct:symlink:opencode:command:close-wkt +/.opencode/commands/close-wkt.md +# END ai.skillz: direct:symlink:opencode:command:close-wkt + +# BEGIN ai.skillz: direct:symlink:opencode:command:dep-supersede-scan +/.opencode/commands/dep-supersede-scan.md +# END ai.skillz: direct:symlink:opencode:command:dep-supersede-scan + +# BEGIN ai.skillz: direct:symlink:opencode:command:harness-perf +/.opencode/commands/harness-perf.md +# END ai.skillz: direct:symlink:opencode:command:harness-perf + +# BEGIN ai.skillz: direct:symlink:opencode:command:yt-url-lookup +/.opencode/commands/yt-url-lookup.md +# END ai.skillz: direct:symlink:opencode:command:yt-url-lookup +# BEGIN ai.skillz: direct:symlink:agents:gish +/.agents/skills/gish +# END ai.skillz: direct:symlink:agents:gish + +# BEGIN ai.skillz: direct:symlink:agents:resolve-conflicts +/.agents/skills/resolve-conflicts +# END ai.skillz: direct:symlink:agents:resolve-conflicts + +# BEGIN ai.skillz: direct:symlink:agents:git-mgmt +/.agents/skills/git-mgmt +# END ai.skillz: direct:symlink:agents:git-mgmt + +# BEGIN ai.skillz: direct:symlink:agents:open-wkt +/.agents/skills/open-wkt +# END ai.skillz: direct:symlink:agents:open-wkt + +# BEGIN ai.skillz: direct:symlink:agents:run-tests +/.agents/skills/run-tests +# END ai.skillz: direct:symlink:agents:run-tests + +# BEGIN ai.skillz: direct:symlink:agents:close-wkt +/.agents/skills/close-wkt +# END ai.skillz: direct:symlink:agents:close-wkt + +# BEGIN ai.skillz: direct:symlink:agents:code-review +/.agents/skills/code-review +# END ai.skillz: direct:symlink:agents:code-review + +# BEGIN ai.skillz: direct:symlink:agents:code-nav-refs +/.agents/skills/code-nav-refs +# END ai.skillz: direct:symlink:agents:code-nav-refs + +# BEGIN ai.skillz: direct:symlink:agents:code-review-changes +/.agents/skills/code-review-changes +# END ai.skillz: direct:symlink:agents:code-review-changes + +# BEGIN ai.skillz: direct:symlink:agents:commit-msg +/.agents/skills/commit-msg +# END ai.skillz: direct:symlink:agents:commit-msg + +# BEGIN ai.skillz: direct:symlink:agents:commit-plan +/.agents/skills/commit-plan +# END ai.skillz: direct:symlink:agents:commit-plan + +# BEGIN ai.skillz: direct:symlink:agents:dep-supersede-scan +/.agents/skills/dep-supersede-scan +# END ai.skillz: direct:symlink:agents:dep-supersede-scan + +# BEGIN ai.skillz: direct:symlink:agents:harness-perf +/.agents/skills/harness-perf +# END ai.skillz: direct:symlink:agents:harness-perf + +# BEGIN ai.skillz: direct:symlink:agents:inter-skill-review +/.agents/skills/inter-skill-review +# END ai.skillz: direct:symlink:agents:inter-skill-review + +# BEGIN ai.skillz: direct:symlink:agents:opencode-cleaning +/.agents/skills/opencode-cleaning +# END ai.skillz: direct:symlink:agents:opencode-cleaning + +# BEGIN ai.skillz: direct:symlink:agents:plan-io +/.agents/skills/plan-io +# END ai.skillz: direct:symlink:agents:plan-io + +# BEGIN ai.skillz: direct:symlink:agents:pr-msg +/.agents/skills/pr-msg +# END ai.skillz: direct:symlink:agents:pr-msg + +# BEGIN ai.skillz: direct:symlink:agents:prompt-io +/.agents/skills/prompt-io +# END ai.skillz: direct:symlink:agents:prompt-io + +# BEGIN ai.skillz: direct:symlink:agents:py-codestyle +/.agents/skills/py-codestyle +# END ai.skillz: direct:symlink:agents:py-codestyle + +# BEGIN ai.skillz: direct:symlink:agents:taken-export +/.agents/skills/taken-export +# END ai.skillz: direct:symlink:agents:taken-export + +# BEGIN ai.skillz: direct:symlink:agents:yt-url-lookup +/.agents/skills/yt-url-lookup +# END ai.skillz: direct:symlink:agents:yt-url-lookup + +# BEGIN ai.skillz: workflow-state +/.ai/state/ +/.ai/commit-msg/conf.toml +/.ai/pr-msg/conf.toml +# END ai.skillz: workflow-state diff --git a/ai/prompt-io/opencode/20260907T214000Z_9115d24_prompt_io.md b/ai/prompt-io/opencode/20260907T214000Z_9115d24_prompt_io.md new file mode 100644 index 00000000..d6008155 --- /dev/null +++ b/ai/prompt-io/opencode/20260907T214000Z_9115d24_prompt_io.md @@ -0,0 +1,34 @@ +--- +model: openai/gpt-5.6-sol +service: opencode +session: unavailable +timestamp: 2026-09-07T21:40:00Z +git_ref: 9115d24 +scope: config +substantive: true +raw_file: 20260907T214000Z_9115d24_prompt_io.raw.md +--- + +## Prompt + +Deploy the provider-neutral `.agents` skill tree for Codex first and +then migrate Piker's workflow state without losing archived helpers. + +## Response summary + +Deployed 21 ignored `.agents/skills` links, copied 132 legacy runtime +files into `.ai/state`, selected the neutral backend, and preserved the +original archive and staged `ruff.toml` change. + +## Files changed + +- `.gitignore` - ignore shared links and generated neutral state. +- `.ai/workflow-state.json` - select the neutral backend. +- `.ai/commit-msg/style-guide-reference.md` - port commit guidance. +- `.ai/run-tests/test-harness-reference.md` - port test guidance. + +## Human edits + +The human prioritized immediate Codex deployment and directed helper +archives to migrate as opaque runtime data instead of blocking the +operation. No direct source-line edits were made by the human. diff --git a/ai/prompt-io/opencode/20260907T214000Z_9115d24_prompt_io.raw.md b/ai/prompt-io/opencode/20260907T214000Z_9115d24_prompt_io.raw.md new file mode 100644 index 00000000..e5a89ef2 --- /dev/null +++ b/ai/prompt-io/opencode/20260907T214000Z_9115d24_prompt_io.raw.md @@ -0,0 +1,15 @@ +--- +model: openai/gpt-5.6-sol +service: opencode +timestamp: 2026-09-07T21:40:00Z +git_ref: 9115d24 +diff_cmd: git diff HEAD~1..HEAD +--- + +Deploy all shared `ai.skillz` skills through `.agents/skills` for +Codex, then migrate Piker's generated workflow state from the legacy +Claude layout to provider-neutral `.ai/state` storage. + +Preserve every legacy source and the existing staged `ruff.toml` +change. Commit only the neutral backend selection, repository guidance, +ignore rules, and this provenance record. diff --git a/ai/prompt-io/opencode/20260908T043134Z_f148fbcc_prompt_io.md b/ai/prompt-io/opencode/20260908T043134Z_f148fbcc_prompt_io.md new file mode 100644 index 00000000..95251360 --- /dev/null +++ b/ai/prompt-io/opencode/20260908T043134Z_f148fbcc_prompt_io.md @@ -0,0 +1,37 @@ +--- +model: openai/gpt-5.6-sol +service: opencode +session: unavailable +timestamp: 2026-09-08T04:31:34Z +git_ref: f148fbcc +scope: config +substantive: true +raw_file: 20260908T043134Z_f148fbcc_prompt_io.raw.md +--- + +## Prompt + +Audit tracked legacy skill files under `.claude/`, then migrate Piker's +repository-specific skills into the provider-neutral discovery tree. +Keep Claude compatibility and remove stale Claude-local configuration. + +## Response summary + +Moves four Piker-specific skill packages to `.agents/skills/`, retains +Claude discovery through relative adapter links, and drops two stale +tracked Claude-local files. + +## Files changed + +- `.agents/skills/piker-*/` - provider-neutral Piker skill sources. +- `.claude/skills/piker-*` - relative Claude compatibility links. +- `.claude/settings.local.json` - remove tracked local permissions. +- `.claude/skills/commit-msg/style-guide-reference.md` - remove the + superseded legacy copy now represented under `.ai/commit-msg/`. + +## Human edits + +The human identified that repository-specific skills were not migrated +by the shared `ai.skillz` deployment, selected `.agents/skills/` as the +neutral source, and requested that the migration land in PR 93 before +following up on the generic migration tooling. diff --git a/ai/prompt-io/opencode/20260908T043134Z_f148fbcc_prompt_io.raw.md b/ai/prompt-io/opencode/20260908T043134Z_f148fbcc_prompt_io.raw.md new file mode 100644 index 00000000..3af350df --- /dev/null +++ b/ai/prompt-io/opencode/20260908T043134Z_f148fbcc_prompt_io.raw.md @@ -0,0 +1,34 @@ +--- +model: openai/gpt-5.6-sol +service: opencode +timestamp: 2026-09-08T04:31:34Z +git_ref: f148fbcc +diff_cmd: git diff --cached +--- + +## Prompt + +Audit tracked legacy skill files under `.claude/`, then migrate Piker's +repository-specific skills into the provider-neutral discovery tree. +Keep Claude compatibility and remove stale Claude-local configuration. + +## Generated changes + +> `git diff --cached -- .agents/skills/` + +Moves the four Piker-owned skill packages into the neutral +`.agents/skills/` source tree without changing their contents. + +> `git diff --cached -- .claude/skills/ .claude/settings.local.json` + +Adds relative Claude adapter links to the neutral skill sources and +removes the tracked local settings file and superseded commit-message +style guide. + +## Validation + +- All four Claude adapter links resolve to a `SKILL.md`. +- `git diff --cached --check` passes. +- No tracked files match the repository's ignore rules. +- Shared deployment validation retains one unrelated stale local + `commit-plan` hybrid-link error for an `ai.skillz` follow-up.