[PR AUDIT] Master: ponytail upstream PRs relevant to Rust port (17 identified)

[getappz]

Source

Audit of 97 open PRs at https://github.com/DietrichGebert/ponytail/pulls (2026-07-07).

Summary

~80 PRs are editor plugins, docs, i18n, npm build fixes — JS-specific, irrelevant.
~17 PRs touch core logic — hooks, personas, review, instructions, modes, error handling.

Architecture/Bug Issues — Rust port must address

#	Upstream PR	What	Rust file
1	ponytail#507	agent_type regex matcher on SubagentStart hook	src/hook.rs
2	ponytail#508	SessionStart additionalContext at top level for Codex	src/hook.rs, crates/ponytail/src/platform.rs
3	ponytail#537	CRLF handling + MCP version from package.json	cross-platform + src/mcp_server.rs
4	ponytail#291	AGENTS.md as fallback, not hardcoded duplicate	crates/ponytail/src/instructions.rs
5	ponytail#303	Log errors to stderr, never swallow silently	all files
6	ponytail#214	Windows stdin hang — hook never blocks on stdin read	src/hook.rs
7	ponytail#517	Detect compression-plugin conflicts	config/startup

Feature Issues — Rust port should design for

#	Upstream PR	What	Rust file
8	ponytail#540	Regex-based over-engineering pre-filter for review	crates/ponytail/src/sub_skills.rs
9	ponytail#524	Review scoping: uncommitted changes or branch diff	sub_skills.rs
10	ponytail#532	Persona wording: "lazy = less work for same result"	instructions.rs
11	ponytail#504	Anti-hallucination companion skill	instructions.rs
12	ponytail#456	ponytail-playbook skill variant (TDD)	sub_skills.rs
13	ponytail#397	Custom skills + mode shortcuts	sub_skills.rs, config.rs
14	ponytail#229	Per-session mode (not just global flag file)	state.rs
15	ponytail#99	Bare /ponytail reports active mode	switcher.rs
16	ponytail#491	Tighten simplification marker rule	instructions.rs
17	ponytail#523	Number review and audit findings	sub_skills.rs
18	ponytail#521	Keep persona out of replies	instructions.rs

[getappz]

Status update

Audited all 18 items against the current codebase. Result: 10 already done, 1 (#3's CRLF portion) doesn't apply to the Rust port, and the remaining 7 are addressed in #96.

#	Upstream PR	Status
1	ponytail#507	✅ Done — agent_type matcher on SubagentStart
2	ponytail#508	✅ Done — Codex additionalContext at top level
3	ponytail#537	⚠️ Partial — MCP version done; CRLF part N/A (JS-specific test-harness bug)
4	ponytail#291	✅ Done — AGENTS.md fallback, not hardcoded duplicate
5	ponytail#303	🔧 Fixed in #96 — src/state.rs no longer silently swallows I/O errors
6	ponytail#214	✅ Done — non-blocking stdin read with timeout
7	ponytail#517	🔧 Fixed in #96 — compression-plugin conflict detection + deconfliction note
8	ponytail#540	✅ Done — over-engineering pre-filter table
9	ponytail#524	✅ Done — review scoping (uncommitted/branch/repo)
10	ponytail#532	✅ Done — "lazy = efficient, not careless" wording
11	ponytail#504	🔧 Fixed in #96 — new ponytail-no-hallucination companion skill
12	ponytail#456	✅ Done — ponytail-playbook TDD skill
13	ponytail#397	✅ Done — custom skills + mode shortcuts
14	ponytail#229	✅ Done — per-session mode, separate from global flag file
15	ponytail#99	🔧 Fixed in #96 — bare /ponytail now reports active mode instead of no-op
16	ponytail#491	🔧 Fixed in #96 — tightened simplification-marker rule
17	ponytail#523	✅ Done — numbered review/audit findings
18	ponytail#521	🔧 Fixed in #96 — persona-boundary rule keeping ponytail/lazy out of replies

Root cause behind #11/#16/#18 all being missed: the correct wording existed only in a dead fallback_instructions() function in crates/ponytail/src/instructions.rs that build() never called, so it never reached the shipped skill.md. #96 moves that content into the live skill and deletes the dead function.

PR: #96

[getappz]

All items resolved — 10 were already done, 1 didn't apply (CRLF portion of #3), and the remaining 7 shipped in #96 (bare /ponytail report, compression-plugin deconfliction, no-hallucination companion skill, tightened simplification-marker rule, persona-boundary rule, state.rs error logging). Closing.