Skip to content

session: regenerate the session-start hook; note the two remote-session lines - #295

Merged
Jammy2211 merged 1 commit into
mainfrom
claude/mobile-performance-check-s8fyel
Aug 26, 2026
Merged

Jammy2211 merged 1 commit into
mainfrom
claude/mobile-performance-check-s8fyel

Conversation

@Jammy2211

Copy link
Copy Markdown
Contributor

Summary

The generated copy of PyAutoMind's canonical policy/session_start_hook.sh, regenerated by repos_sync.py --write after the multi-repo session fixes in PyAutoLabs/PyAutoMind#342. Source of truth is Mind; this repo carries a copy because Claude Code reads hooks per repo.

What changed upstream, in one line each: pytest-xdist joins the session venv's base deps (this repo's 554 tests go from 96s on one core to 28s on four); /usr/local/bin/python{,3} become wrapper scripts backed by that venv rather than symlinks to the base interpreter, so python3 -m pytest works in a session that got no PATH export; uv's isolated pytest is pointed at the venv's, so bare pytest stops failing collection on yaml; and the workspace-root fan-out — previously unreachable by construction — now installs from any session.

AGENTS.md gains two measured lines. This file is auto-loaded in every session and a remote one has little else to go on, so it is the right place for them: run the suite with -n auto, and knock on session_bootstrap.sh in the first turn, since a session holding several organs registers no SessionStart hook at all.

API Changes

None — internal changes only. .claude/hooks/session-start.sh is a generated file (PyAutoMind/scripts/repos_sync.py --write), not an interface this repo owns; AGENTS.md is prose. No entrypoint, verb, or agent signature changes.

Test Plan

Merge order

No gate between this PR and PyAutoLabs/PyAutoMind#342. firewall_gate.yml is path-filtered to scripts/repos_sync.py, which neither PR touches, so the Brain-before-Mind order a previous session needed does not apply. Merging Mind first keeps source ahead of its generated copy, but either order is safe.

Notes for the reviewer

Gate: Heart is unreachable in this session (PyAutoHeart not checked out), so ship_library.md step 3's documented fallback was used — per-repo pytest -x, any failure treated as RED. 554 tests pass here, 232 in PyAutoMind.

Known drift, not fixed here: PyAutoHeart and PyAutoHands carry the same generated copy. repos_sync.py --write regenerated two of four because only two repos are attached to this session. They need the same one-file update.

Generated by the PyAutoLabs agent workflow.


Generated by Claude Code

…on lines

The generated copy of PyAutoMind's canonical `policy/session_start_hook.sh`,
regenerated by `repos_sync.py --write` after the multi-repo session fixes
(pytest-xdist in the base deps, a venv-backed `python3` wrapper rather than a
symlink to the base interpreter, a reachable workspace-root fan-out).

AGENTS.md gains two measured lines, because this file is loaded in every
session and a remote one has little else to go on: run the suite with
`-n auto` (554 tests, 96s on one core and 28s on four), and knock on
`session_bootstrap.sh` in the first turn, since a session holding several
organs registers no SessionStart hook at all.

Validation: 554 tests pass, `repos_sync.py --check` clean against this copy.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XLmJCBKNoh2tB5UkdecsSn
@Jammy2211
Jammy2211 merged commit ab79451 into main Aug 26, 2026
2 checks passed
@Jammy2211
Jammy2211 deleted the claude/mobile-performance-check-s8fyel branch August 26, 2026 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants