fix(skills): use script-backed ai-memory commands#110
Closed
thecontstruct wants to merge 2 commits intoHidden-History:mainfrom
Closed
fix(skills): use script-backed ai-memory commands#110thecontstruct wants to merge 2 commits intoHidden-History:mainfrom
thecontstruct wants to merge 2 commits intoHidden-History:mainfrom
Conversation
Move the Parzival and status skill runtimes into real scripts and update skill/workflow docs to point at the installed run-with-env wrapper. This keeps command examples accurate and prevents skill execution from silently using the wrong Python environment. Made-with: Cursor
Hidden-History
pushed a commit
that referenced
this pull request
Apr 13, 2026
…ded PR #110) Adds 3 new Python scripts and updates 9 existing skills + 3 workflow docs to use the canonical run-with-env.sh wrapper pattern for hook bootstrap. New scripts (scripts/memory/): - aim_status.py — status reporting via installed ai-memory wrapper - parzival_save_handoff.py — handoff save path invocable via wrapper - parzival_save_insight.py — insight save path invocable via wrapper Updated skills (_ai-memory/skills/): aim-github-search, aim-jira-search, aim-jira-sync, aim-save, aim-search, aim-settings, aim-status, parzival-save-handoff, parzival-save-insight — all switched to run-with-env.sh script wrapper invocation. Updated workflow docs (_ai-memory/pov/workflows/session/): close/instructions.md, close/steps-c/step-04-save-and-confirm.md, handoff/steps-c/step-03-update-index.md — documentation of the wrapper invocation pattern for closeout and handoff. Originally submitted as PR #110 on phil/ai-memory-venv-and-qdrant-wrapper-fix. Cherry-picked directly to main from test/phase-b-verify (live-verified 9/9 stages in PM #248) after PR #111's squash-merge superseded PR #110's older versions of scripts/memory/run-with-env.sh and _ai-memory/skills/aim-github-sync/SKILL.md. Content is byte-identical to the tested state. Co-Authored-By: Phil Mahncke <pmahncke@axonify.com> Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Owner
|
Superseded by PR #111 (run-with-env.sh quote-strip fix) + direct cherry-pick to main at commit 4b1bb15 with Co-Authored-By attribution. Content cherry-picked is byte-identical to the test/phase-b-verify state that was live-verified 9/9 in PM #248. Thanks @phil-nguyen for the contribution. Closing as merged-equivalent. |
Hidden-History
pushed a commit
that referenced
this pull request
Apr 13, 2026
- Add CHANGELOG [2.3.2] entry (2026-04-13) - Bump version 2.3.1 → 2.3.2: pyproject.toml, __version__.py, README.md badge, docker/embedding/main.py, metrics.py fallback, version.txt (stale since v2.0.6 — corrected) Scope: - Security: 14 vulnerability alerts closed — pygments ReDoS (GHSA-5239-wwwm-4pmq), anthropic SDK (GHSA-w828-4qhx-vxx3, GHSA-q5f5-3gjm-7mfm), pytest (CVE-2025-71176), black HIGH auto-resolved via uv.lock regeneration (PRs #99, #105, #107, #112) - Added: group_id normalization + audit/migrate tools (PR #111); run-with-env.sh canonical wrapper + script-backed skills (PR #110, cherry-picked as 4b1bb15) - Fixed: 4 Phase B live-verify regressions on PR #111 branch — .env quote stripping, install_dir catastrophic regression in github-sync container, AliasChoices binding for install_dir, trace_buffer volume missing from github-sync service Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
scripts/memory/*.pyentrypoints so skill execution no longer depends on implicit inline Pythonrun-with-env.shthe canonical wrapper and update skill/workflow docs to use the installed ai-memory path with clearer activation vs shell execution examplesTest plan
python3 -m py_compile scripts/memory/parzival_save_handoff.py scripts/memory/parzival_save_insight.py scripts/memory/aim_status.pyshellcheck scripts/memory/run-with-env.sh./scripts/memory/run-with-env.sh aim_status.py --section flags./scripts/memory/run-with-env.sh parzival_save_insight.py "pre-pr repo smoke ..."./scripts/memory/run-with-env.sh parzival_save_handoff.py --file <tempfile>~/.ai-memory/scripts/memory/run-with-env.sh aim_status.py --section flags~/.ai-memory/scripts/memory/run-with-env.sh parzival_save_insight.py "true installed smoke ..."~/.ai-memory/scripts/memory/run-with-env.sh parzival_save_handoff.py --file <tempfile>Made with Cursor