chore(ci): narrow the Effect conventions check-run agent

[juliusmarminge]

Follow-up to #9297 and #9298, same treatment for the Effect agent. It isn't the cost problem the UI agent was, but it had the same drift in smaller doses plus one scoping mistake.

Runs on PRs it can't act on.include listed **/*.tsx, but Effect services (Context.Service, Layer.*, Schema.TaggedErrorClass) live in 258 .ts files across the repo and zero .tsx files. Every UI-only PR woke the agent to read the diff and say All clear. Now .ts only, with **/*.test.ts excluded since the prompt already carves tests out of the injection rules.

Stale, PR-specific rules. Removed the startup-reconciliation rule from #7719 (the design of one feature, not a convention) and the domain/Services + domain/Layers → domain/Foo.ts migration section (those directories no longer exist). The general "delete old files, update every consumer, no re-export shims" rule survives as one line.

Flagging old code in touched files. Reporting said "introduced or retained in the pull request's changed scope," which had the agent dinging pre-existing FooShape interfaces (40 remain) and Effect.catchTag uses (8 files) in any file a PR touched. Now "introduced by changed lines," matching the UI agent.

Repetition. The errors section had eight bullets that each restated some mix of "derive message from structural fields, never cause; keep cause; no detail copying cause.message; no raw payloads." Folded into six with every rule preserved.

Also dropped github_api_read_only (unused) and git_tools (redundant with browse_code). Kept claude-opus-5 and effort: high.

The catchTags-only rule stays as is. Worth knowing that .repos/effect-smol/LLMS.md, which AGENTS.md points agents at before writing Effect code, uses Effect.catchTag in its examples, so agents following the reference will write code this check fails. If that preference isn't deliberate, dropping the ban is a one-line follow-up.

Claude Fable 5 via Claude Code

🤖 Generated with Claude Code

---

Note

Low Risk
Documentation-only change to a CI review agent prompt and triggers; no application runtime or Effect service code is modified.

Overview
Tightens the Effect Service Conventions Macroscope check-run agent so it runs less often, reviews less code, and flags fewer false positives.

Scope and tooling:include is .ts only (drops **/*.tsx because Effect services aren’t in TSX) and adds exclude: **/*.test.ts. Removes unused github_api_read_only and redundant git_tools (keeps browse_code).

Prompt behavior: Review is limited to lines the PR actually changed; legacy patterns in the same file are out of scope—aligned with the UI consistency agent. Reporting shifts from “introduced or retained” to “introduced by changed lines.”

Rule cleanup: Drops obsolete file-layout/migration guidance (merged into a single “no re-export shims” line when services move), removes the startup-reconciliation error rule, and condenses the errors section without dropping the substantive rules. Opening instructions are shortened throughout.

^{Reviewed by Cursor Bugbot for commit 80527dd. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Narrow Effect service conventions check-run agent scope to changed lines

Updates the check-run agent policy in effect-service-conventions.md to reduce review noise and focus on the diff. The agent now excludes **/*.test.ts and all TSX files, no longer uses Git or GitHub read-only tools, and reports only violations on changed lines instead of repository-wide call-site cleanup. Guidance is consolidated and shortened across Effect imports, service definitions, dependency acquisition, runtime boundaries, and errors; added a requirement to delete obsolete service files and compatibility shims during service moves. A clean review must return exactly All clear.

• Behavioral Change: agent scope and tool set change — removed Git/GitHub tools and TSX/test file inclusion may miss issues previously caught in those files.

📊 Macroscope summarized 80527dd. 1 file reviewed, 1 issue evaluated, 0 issues filtered, 1 comment posted

🗂️ Filtered Issues

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.4 KiB	13.2 KiB	−230 B (−1.7%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	0 B (0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.6 KiB	6.3 KiB	−230 B (−3.4%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.0 KiB	55.5 KiB	−1.5 KiB (−2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	8	−2 (−20.0%)	21	✅
Claude	Total thread wire	13.3 KiB	13.3 KiB	+15 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	−4 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	+19 B (+0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	56.4 KiB	56.4 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: 194f838 · PR result: 80527dd · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[juliusmarminge]

Follow-up to #9297 and #9298, same treatment for the Effect agent. It isn't the cost problem the UI agent was, but it had the same drift in smaller doses plus one scoping mistake.

Runs on PRs it can't act on.include listed **/*.tsx, but Effect services (Context.Service, Layer.*, Schema.TaggedErrorClass) live in 258 .ts files across the repo and zero .tsx files. Every UI-only PR woke the agent to read the diff and say All clear. Now .ts only, with **/*.test.ts excluded since the prompt already carves tests out of the injection rules.

Stale, PR-specific rules. Removed the startup-reconciliation rule from #7719 (the design of one feature, not a convention) and the domain/Services + domain/Layers → domain/Foo.ts migration section (those directories no longer exist). The general "delete old files, update every consumer, no re-export shims" rule survives as one line.

Flagging old code in touched files. Reporting said "introduced or retained in the pull request's changed scope," which had the agent dinging pre-existing FooShape interfaces (40 remain) and Effect.catchTag uses (8 files) in any file a PR touched. Now "introduced by changed lines," matching the UI agent.

Repetition. The errors section had eight bullets that each restated some mix of "derive message from structural fields, never cause; keep cause; no detail copying cause.message; no raw payloads." Folded into six with every rule preserved.

Also dropped github_api_read_only (unused) and git_tools (redundant with browse_code). Kept claude-opus-5 and effort: high.

The catchTags-only rule stays as is. Worth knowing that .repos/effect-smol/LLMS.md, which AGENTS.md points agents at before writing Effect code, uses Effect.catchTag in its examples, so agents following the reference will write code this check fails. If that preference isn't deliberate, dropping the ban is a one-line follow-up.

Claude Fable 5 via Claude Code

🤖 Generated with Claude Code

---

Note

Low Risk
Documentation-only change to a CI review agent prompt and triggers; no application runtime or Effect service code is modified.

Overview
Tightens the Effect Service Conventions Macroscope check-run agent so it runs less often, reviews less code, and flags fewer false positives.

Scope and tooling:include is .ts only (drops **/*.tsx because Effect services aren’t in TSX) and adds exclude: **/*.test.ts. Removes unused github_api_read_only and redundant git_tools (keeps browse_code).

Prompt behavior: Review is limited to lines the PR actually changed; legacy patterns in the same file are out of scope—aligned with the UI consistency agent. Reporting shifts from “introduced or retained” to “introduced by changed lines.”

Rule cleanup: Drops obsolete file-layout/migration guidance (merged into a single “no re-export shims” line when services move), removes the startup-reconciliation error rule, and condenses the errors section without dropping the substantive rules. Opening instructions are shortened throughout.

^{Reviewed by Cursor Bugbot for commit 80527dd. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Narrow Effect service conventions check-run agent scope to changed lines

Updates the check-run agent policy in effect-service-conventions.md to reduce review noise and focus on the diff. The agent now excludes **/*.test.ts and all TSX files, no longer uses Git or GitHub read-only tools, and reports only violations on changed lines instead of repository-wide call-site cleanup. Guidance is consolidated and shortened across Effect imports, service definitions, dependency acquisition, runtime boundaries, and errors; added a requirement to delete obsolete service files and compatibility shims during service moves. A clean review must return exactly All clear.

• Behavioral Change: agent scope and tool set change — removed Git/GitHub tools and TSX/test file inclusion may miss issues previously caught in those files.

📊 Macroscope summarized 80527dd. 1 file reviewed, 1 issue evaluated, 0 issues filtered, 1 comment posted

🗂️ Filtered Issues

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.4 KiB	13.2 KiB	−230 B (−1.7%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	0 B (0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.6 KiB	6.3 KiB	−230 B (−3.4%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.0 KiB	55.5 KiB	−1.5 KiB (−2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	8	−2 (−20.0%)	21	✅
Claude	Total thread wire	13.3 KiB	13.3 KiB	+15 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	−4 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	+19 B (+0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	56.4 KiB	56.4 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: 194f838 · PR result: 80527dd · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[juliusmarminge]

Follow-up to #9297 and #9298, same treatment for the Effect agent. It isn't the cost problem the UI agent was, but it had the same drift in smaller doses plus one scoping mistake.

Runs on PRs it can't act on.include listed **/*.tsx, but Effect services (Context.Service, Layer.*, Schema.TaggedErrorClass) live in 258 .ts files across the repo and zero .tsx files. Every UI-only PR woke the agent to read the diff and say All clear. Now .ts only, with **/*.test.ts excluded since the prompt already carves tests out of the injection rules.

Stale, PR-specific rules. Removed the startup-reconciliation rule from #7719 (the design of one feature, not a convention) and the domain/Services + domain/Layers → domain/Foo.ts migration section (those directories no longer exist). The general "delete old files, update every consumer, no re-export shims" rule survives as one line.

Flagging old code in touched files. Reporting said "introduced or retained in the pull request's changed scope," which had the agent dinging pre-existing FooShape interfaces (40 remain) and Effect.catchTag uses (8 files) in any file a PR touched. Now "introduced by changed lines," matching the UI agent.

Repetition. The errors section had eight bullets that each restated some mix of "derive message from structural fields, never cause; keep cause; no detail copying cause.message; no raw payloads." Folded into six with every rule preserved.

Also dropped github_api_read_only (unused) and git_tools (redundant with browse_code). Kept claude-opus-5 and effort: high.

The catchTags-only rule stays as is. Worth knowing that .repos/effect-smol/LLMS.md, which AGENTS.md points agents at before writing Effect code, uses Effect.catchTag in its examples, so agents following the reference will write code this check fails. If that preference isn't deliberate, dropping the ban is a one-line follow-up.

Claude Fable 5 via Claude Code

🤖 Generated with Claude Code

---

Note

Low Risk
Documentation-only change to a CI review agent prompt and triggers; no application runtime or Effect service code is modified.

Overview
Tightens the Effect Service Conventions Macroscope check-run agent so it runs less often, reviews less code, and flags fewer false positives.

Scope and tooling:include is .ts only (drops **/*.tsx because Effect services aren’t in TSX) and adds exclude: **/*.test.ts. Removes unused github_api_read_only and redundant git_tools (keeps browse_code).

Prompt behavior: Review is limited to lines the PR actually changed; legacy patterns in the same file are out of scope—aligned with the UI consistency agent. Reporting shifts from “introduced or retained” to “introduced by changed lines.”

Rule cleanup: Drops obsolete file-layout/migration guidance (merged into a single “no re-export shims” line when services move), removes the startup-reconciliation error rule, and condenses the errors section without dropping the substantive rules. Opening instructions are shortened throughout.

^{Reviewed by Cursor Bugbot for commit 80527dd. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Narrow Effect service conventions check-run agent scope to changed lines

Updates the check-run agent policy in effect-service-conventions.md to reduce review noise and focus on the diff. The agent now excludes **/*.test.ts and all TSX files, no longer uses Git or GitHub read-only tools, and reports only violations on changed lines instead of repository-wide call-site cleanup. Guidance is consolidated and shortened across Effect imports, service definitions, dependency acquisition, runtime boundaries, and errors; added a requirement to delete obsolete service files and compatibility shims during service moves. A clean review must return exactly All clear.

• Behavioral Change: agent scope and tool set change — removed Git/GitHub tools and TSX/test file inclusion may miss issues previously caught in those files.

📊 Macroscope summarized 80527dd. 1 file reviewed, 1 issue evaluated, 0 issues filtered, 1 comment posted

🗂️ Filtered Issues

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.4 KiB	13.2 KiB	−230 B (−1.7%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	0 B (0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.6 KiB	6.3 KiB	−230 B (−3.4%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.0 KiB	55.5 KiB	−1.5 KiB (−2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	8	−2 (−20.0%)	21	✅
Claude	Total thread wire	13.3 KiB	13.3 KiB	+15 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	−4 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	+19 B (+0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	56.4 KiB	56.4 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: 194f838 · PR result: 80527dd · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[juliusmarminge]

Follow-up to #9297 and #9298, same treatment for the Effect agent. It isn't the cost problem the UI agent was, but it had the same drift in smaller doses plus one scoping mistake.

Runs on PRs it can't act on.include listed **/*.tsx, but Effect services (Context.Service, Layer.*, Schema.TaggedErrorClass) live in 258 .ts files across the repo and zero .tsx files. Every UI-only PR woke the agent to read the diff and say All clear. Now .ts only, with **/*.test.ts excluded since the prompt already carves tests out of the injection rules.

Stale, PR-specific rules. Removed the startup-reconciliation rule from #7719 (the design of one feature, not a convention) and the domain/Services + domain/Layers → domain/Foo.ts migration section (those directories no longer exist). The general "delete old files, update every consumer, no re-export shims" rule survives as one line.

Flagging old code in touched files. Reporting said "introduced or retained in the pull request's changed scope," which had the agent dinging pre-existing FooShape interfaces (40 remain) and Effect.catchTag uses (8 files) in any file a PR touched. Now "introduced by changed lines," matching the UI agent.

Repetition. The errors section had eight bullets that each restated some mix of "derive message from structural fields, never cause; keep cause; no detail copying cause.message; no raw payloads." Folded into six with every rule preserved.

Also dropped github_api_read_only (unused) and git_tools (redundant with browse_code). Kept claude-opus-5 and effort: high.

The catchTags-only rule stays as is. Worth knowing that .repos/effect-smol/LLMS.md, which AGENTS.md points agents at before writing Effect code, uses Effect.catchTag in its examples, so agents following the reference will write code this check fails. If that preference isn't deliberate, dropping the ban is a one-line follow-up.

Claude Fable 5 via Claude Code

🤖 Generated with Claude Code

---

Note

Low Risk
Documentation-only change to a CI review agent prompt and triggers; no application runtime or Effect service code is modified.

Overview
Tightens the Effect Service Conventions Macroscope check-run agent so it runs less often, reviews less code, and flags fewer false positives.

Scope and tooling:include is .ts only (drops **/*.tsx because Effect services aren’t in TSX) and adds exclude: **/*.test.ts. Removes unused github_api_read_only and redundant git_tools (keeps browse_code).

Prompt behavior: Review is limited to lines the PR actually changed; legacy patterns in the same file are out of scope—aligned with the UI consistency agent. Reporting shifts from “introduced or retained” to “introduced by changed lines.”

Rule cleanup: Drops obsolete file-layout/migration guidance (merged into a single “no re-export shims” line when services move), removes the startup-reconciliation error rule, and condenses the errors section without dropping the substantive rules. Opening instructions are shortened throughout.

^{Reviewed by Cursor Bugbot for commit 80527dd. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Narrow Effect service conventions check-run agent scope to changed lines

Updates the check-run agent policy in effect-service-conventions.md to reduce review noise and focus on the diff. The agent now excludes **/*.test.ts and all TSX files, no longer uses Git or GitHub read-only tools, and reports only violations on changed lines instead of repository-wide call-site cleanup. Guidance is consolidated and shortened across Effect imports, service definitions, dependency acquisition, runtime boundaries, and errors; added a requirement to delete obsolete service files and compatibility shims during service moves. A clean review must return exactly All clear.

• Behavioral Change: agent scope and tool set change — removed Git/GitHub tools and TSX/test file inclusion may miss issues previously caught in those files.

📊 Macroscope summarized 80527dd. 1 file reviewed, 1 issue evaluated, 0 issues filtered, 1 comment posted

🗂️ Filtered Issues

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.4 KiB	13.2 KiB	−230 B (−1.7%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	0 B (0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.6 KiB	6.3 KiB	−230 B (−3.4%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.0 KiB	55.5 KiB	−1.5 KiB (−2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	8	−2 (−20.0%)	21	✅
Claude	Total thread wire	13.3 KiB	13.3 KiB	+15 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	−4 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	+19 B (+0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	56.4 KiB	56.4 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: 194f838 · PR result: 80527dd · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[juliusmarminge]

Follow-up to #9297 and #9298, same treatment for the Effect agent. It isn't the cost problem the UI agent was, but it had the same drift in smaller doses plus one scoping mistake.

Runs on PRs it can't act on.include listed **/*.tsx, but Effect services (Context.Service, Layer.*, Schema.TaggedErrorClass) live in 258 .ts files across the repo and zero .tsx files. Every UI-only PR woke the agent to read the diff and say All clear. Now .ts only, with **/*.test.ts excluded since the prompt already carves tests out of the injection rules.

Stale, PR-specific rules. Removed the startup-reconciliation rule from #7719 (the design of one feature, not a convention) and the domain/Services + domain/Layers → domain/Foo.ts migration section (those directories no longer exist). The general "delete old files, update every consumer, no re-export shims" rule survives as one line.

Flagging old code in touched files. Reporting said "introduced or retained in the pull request's changed scope," which had the agent dinging pre-existing FooShape interfaces (40 remain) and Effect.catchTag uses (8 files) in any file a PR touched. Now "introduced by changed lines," matching the UI agent.

Repetition. The errors section had eight bullets that each restated some mix of "derive message from structural fields, never cause; keep cause; no detail copying cause.message; no raw payloads." Folded into six with every rule preserved.

Also dropped github_api_read_only (unused) and git_tools (redundant with browse_code). Kept claude-opus-5 and effort: high.

The catchTags-only rule stays as is. Worth knowing that .repos/effect-smol/LLMS.md, which AGENTS.md points agents at before writing Effect code, uses Effect.catchTag in its examples, so agents following the reference will write code this check fails. If that preference isn't deliberate, dropping the ban is a one-line follow-up.

Claude Fable 5 via Claude Code

🤖 Generated with Claude Code

---

Note

Low Risk
Documentation-only change to a CI review agent prompt and triggers; no application runtime or Effect service code is modified.

Overview
Tightens the Effect Service Conventions Macroscope check-run agent so it runs less often, reviews less code, and flags fewer false positives.

Scope and tooling:include is .ts only (drops **/*.tsx because Effect services aren’t in TSX) and adds exclude: **/*.test.ts. Removes unused github_api_read_only and redundant git_tools (keeps browse_code).

Prompt behavior: Review is limited to lines the PR actually changed; legacy patterns in the same file are out of scope—aligned with the UI consistency agent. Reporting shifts from “introduced or retained” to “introduced by changed lines.”

Rule cleanup: Drops obsolete file-layout/migration guidance (merged into a single “no re-export shims” line when services move), removes the startup-reconciliation error rule, and condenses the errors section without dropping the substantive rules. Opening instructions are shortened throughout.

^{Reviewed by Cursor Bugbot for commit 80527dd. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Narrow Effect service conventions check-run agent scope to changed lines

Updates the check-run agent policy in effect-service-conventions.md to reduce review noise and focus on the diff. The agent now excludes **/*.test.ts and all TSX files, no longer uses Git or GitHub read-only tools, and reports only violations on changed lines instead of repository-wide call-site cleanup. Guidance is consolidated and shortened across Effect imports, service definitions, dependency acquisition, runtime boundaries, and errors; added a requirement to delete obsolete service files and compatibility shims during service moves. A clean review must return exactly All clear.

• Behavioral Change: agent scope and tool set change — removed Git/GitHub tools and TSX/test file inclusion may miss issues previously caught in those files.

📊 Macroscope summarized 80527dd. 1 file reviewed, 1 issue evaluated, 0 issues filtered, 1 comment posted

🗂️ Filtered Issues

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.4 KiB	13.2 KiB	−230 B (−1.7%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	0 B (0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.6 KiB	6.3 KiB	−230 B (−3.4%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.0 KiB	55.5 KiB	−1.5 KiB (−2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	8	−2 (−20.0%)	21	✅
Claude	Total thread wire	13.3 KiB	13.3 KiB	+15 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	−4 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	+19 B (+0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	56.4 KiB	56.4 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: 194f838 · PR result: 80527dd · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[juliusmarminge]

Follow-up to #9297 and #9298, same treatment for the Effect agent. It isn't the cost problem the UI agent was, but it had the same drift in smaller doses plus one scoping mistake.

Runs on PRs it can't act on.include listed **/*.tsx, but Effect services (Context.Service, Layer.*, Schema.TaggedErrorClass) live in 258 .ts files across the repo and zero .tsx files. Every UI-only PR woke the agent to read the diff and say All clear. Now .ts only, with **/*.test.ts excluded since the prompt already carves tests out of the injection rules.

Stale, PR-specific rules. Removed the startup-reconciliation rule from #7719 (the design of one feature, not a convention) and the domain/Services + domain/Layers → domain/Foo.ts migration section (those directories no longer exist). The general "delete old files, update every consumer, no re-export shims" rule survives as one line.

Flagging old code in touched files. Reporting said "introduced or retained in the pull request's changed scope," which had the agent dinging pre-existing FooShape interfaces (40 remain) and Effect.catchTag uses (8 files) in any file a PR touched. Now "introduced by changed lines," matching the UI agent.

Repetition. The errors section had eight bullets that each restated some mix of "derive message from structural fields, never cause; keep cause; no detail copying cause.message; no raw payloads." Folded into six with every rule preserved.

Also dropped github_api_read_only (unused) and git_tools (redundant with browse_code). Kept claude-opus-5 and effort: high.

The catchTags-only rule stays as is. Worth knowing that .repos/effect-smol/LLMS.md, which AGENTS.md points agents at before writing Effect code, uses Effect.catchTag in its examples, so agents following the reference will write code this check fails. If that preference isn't deliberate, dropping the ban is a one-line follow-up.

Claude Fable 5 via Claude Code

🤖 Generated with Claude Code

---

Note

Low Risk
Documentation-only change to a CI review agent prompt and triggers; no application runtime or Effect service code is modified.

Overview
Tightens the Effect Service Conventions Macroscope check-run agent so it runs less often, reviews less code, and flags fewer false positives.

Scope and tooling:include is .ts only (drops **/*.tsx because Effect services aren’t in TSX) and adds exclude: **/*.test.ts. Removes unused github_api_read_only and redundant git_tools (keeps browse_code).

Prompt behavior: Review is limited to lines the PR actually changed; legacy patterns in the same file are out of scope—aligned with the UI consistency agent. Reporting shifts from “introduced or retained” to “introduced by changed lines.”

Rule cleanup: Drops obsolete file-layout/migration guidance (merged into a single “no re-export shims” line when services move), removes the startup-reconciliation error rule, and condenses the errors section without dropping the substantive rules. Opening instructions are shortened throughout.

^{Reviewed by Cursor Bugbot for commit 80527dd. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Narrow Effect service conventions check-run agent scope to changed lines

Updates the check-run agent policy in effect-service-conventions.md to reduce review noise and focus on the diff. The agent now excludes **/*.test.ts and all TSX files, no longer uses Git or GitHub read-only tools, and reports only violations on changed lines instead of repository-wide call-site cleanup. Guidance is consolidated and shortened across Effect imports, service definitions, dependency acquisition, runtime boundaries, and errors; added a requirement to delete obsolete service files and compatibility shims during service moves. A clean review must return exactly All clear.

• Behavioral Change: agent scope and tool set change — removed Git/GitHub tools and TSX/test file inclusion may miss issues previously caught in those files.

📊 Macroscope summarized 80527dd. 1 file reviewed, 1 issue evaluated, 0 issues filtered, 1 comment posted

🗂️ Filtered Issues

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.4 KiB	13.2 KiB	−230 B (−1.7%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	0 B (0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.6 KiB	6.3 KiB	−230 B (−3.4%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.0 KiB	55.5 KiB	−1.5 KiB (−2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	8	−2 (−20.0%)	21	✅
Claude	Total thread wire	13.3 KiB	13.3 KiB	+15 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	−4 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	+19 B (+0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	56.4 KiB	56.4 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: 194f838 · PR result: 80527dd · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[juliusmarminge]

Follow-up to #9297 and #9298, same treatment for the Effect agent. It isn't the cost problem the UI agent was, but it had the same drift in smaller doses plus one scoping mistake.

Runs on PRs it can't act on.include listed **/*.tsx, but Effect services (Context.Service, Layer.*, Schema.TaggedErrorClass) live in 258 .ts files across the repo and zero .tsx files. Every UI-only PR woke the agent to read the diff and say All clear. Now .ts only, with **/*.test.ts excluded since the prompt already carves tests out of the injection rules.

Stale, PR-specific rules. Removed the startup-reconciliation rule from #7719 (the design of one feature, not a convention) and the domain/Services + domain/Layers → domain/Foo.ts migration section (those directories no longer exist). The general "delete old files, update every consumer, no re-export shims" rule survives as one line.

Flagging old code in touched files. Reporting said "introduced or retained in the pull request's changed scope," which had the agent dinging pre-existing FooShape interfaces (40 remain) and Effect.catchTag uses (8 files) in any file a PR touched. Now "introduced by changed lines," matching the UI agent.

Repetition. The errors section had eight bullets that each restated some mix of "derive message from structural fields, never cause; keep cause; no detail copying cause.message; no raw payloads." Folded into six with every rule preserved.

Also dropped github_api_read_only (unused) and git_tools (redundant with browse_code). Kept claude-opus-5 and effort: high.

The catchTags-only rule stays as is. Worth knowing that .repos/effect-smol/LLMS.md, which AGENTS.md points agents at before writing Effect code, uses Effect.catchTag in its examples, so agents following the reference will write code this check fails. If that preference isn't deliberate, dropping the ban is a one-line follow-up.

Claude Fable 5 via Claude Code

🤖 Generated with Claude Code

---

Note

Low Risk
Documentation-only change to a CI review agent prompt and triggers; no application runtime or Effect service code is modified.

Overview
Tightens the Effect Service Conventions Macroscope check-run agent so it runs less often, reviews less code, and flags fewer false positives.

Scope and tooling:include is .ts only (drops **/*.tsx because Effect services aren’t in TSX) and adds exclude: **/*.test.ts. Removes unused github_api_read_only and redundant git_tools (keeps browse_code).

Prompt behavior: Review is limited to lines the PR actually changed; legacy patterns in the same file are out of scope—aligned with the UI consistency agent. Reporting shifts from “introduced or retained” to “introduced by changed lines.”

Rule cleanup: Drops obsolete file-layout/migration guidance (merged into a single “no re-export shims” line when services move), removes the startup-reconciliation error rule, and condenses the errors section without dropping the substantive rules. Opening instructions are shortened throughout.

^{Reviewed by Cursor Bugbot for commit 80527dd. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Narrow Effect service conventions check-run agent scope to changed lines

Updates the check-run agent policy in effect-service-conventions.md to reduce review noise and focus on the diff. The agent now excludes **/*.test.ts and all TSX files, no longer uses Git or GitHub read-only tools, and reports only violations on changed lines instead of repository-wide call-site cleanup. Guidance is consolidated and shortened across Effect imports, service definitions, dependency acquisition, runtime boundaries, and errors; added a requirement to delete obsolete service files and compatibility shims during service moves. A clean review must return exactly All clear.

• Behavioral Change: agent scope and tool set change — removed Git/GitHub tools and TSX/test file inclusion may miss issues previously caught in those files.

📊 Macroscope summarized 80527dd. 1 file reviewed, 1 issue evaluated, 0 issues filtered, 1 comment posted

🗂️ Filtered Issues

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.4 KiB	13.2 KiB	−230 B (−1.7%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	0 B (0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.6 KiB	6.3 KiB	−230 B (−3.4%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.0 KiB	55.5 KiB	−1.5 KiB (−2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	8	−2 (−20.0%)	21	✅
Claude	Total thread wire	13.3 KiB	13.3 KiB	+15 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	−4 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	+19 B (+0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	56.4 KiB	56.4 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: 194f838 · PR result: 80527dd · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[juliusmarminge]

Follow-up to #9297 and #9298, same treatment for the Effect agent. It isn't the cost problem the UI agent was, but it had the same drift in smaller doses plus one scoping mistake.

Runs on PRs it can't act on.include listed **/*.tsx, but Effect services (Context.Service, Layer.*, Schema.TaggedErrorClass) live in 258 .ts files across the repo and zero .tsx files. Every UI-only PR woke the agent to read the diff and say All clear. Now .ts only, with **/*.test.ts excluded since the prompt already carves tests out of the injection rules.

Stale, PR-specific rules. Removed the startup-reconciliation rule from #7719 (the design of one feature, not a convention) and the domain/Services + domain/Layers → domain/Foo.ts migration section (those directories no longer exist). The general "delete old files, update every consumer, no re-export shims" rule survives as one line.

Flagging old code in touched files. Reporting said "introduced or retained in the pull request's changed scope," which had the agent dinging pre-existing FooShape interfaces (40 remain) and Effect.catchTag uses (8 files) in any file a PR touched. Now "introduced by changed lines," matching the UI agent.

Repetition. The errors section had eight bullets that each restated some mix of "derive message from structural fields, never cause; keep cause; no detail copying cause.message; no raw payloads." Folded into six with every rule preserved.

Also dropped github_api_read_only (unused) and git_tools (redundant with browse_code). Kept claude-opus-5 and effort: high.

The catchTags-only rule stays as is. Worth knowing that .repos/effect-smol/LLMS.md, which AGENTS.md points agents at before writing Effect code, uses Effect.catchTag in its examples, so agents following the reference will write code this check fails. If that preference isn't deliberate, dropping the ban is a one-line follow-up.

Claude Fable 5 via Claude Code

🤖 Generated with Claude Code

---

Note

Low Risk
Documentation-only change to a CI review agent prompt and triggers; no application runtime or Effect service code is modified.

Overview
Tightens the Effect Service Conventions Macroscope check-run agent so it runs less often, reviews less code, and flags fewer false positives.

Scope and tooling:include is .ts only (drops **/*.tsx because Effect services aren’t in TSX) and adds exclude: **/*.test.ts. Removes unused github_api_read_only and redundant git_tools (keeps browse_code).

Prompt behavior: Review is limited to lines the PR actually changed; legacy patterns in the same file are out of scope—aligned with the UI consistency agent. Reporting shifts from “introduced or retained” to “introduced by changed lines.”

Rule cleanup: Drops obsolete file-layout/migration guidance (merged into a single “no re-export shims” line when services move), removes the startup-reconciliation error rule, and condenses the errors section without dropping the substantive rules. Opening instructions are shortened throughout.

^{Reviewed by Cursor Bugbot for commit 80527dd. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Narrow Effect service conventions check-run agent scope to changed lines

Updates the check-run agent policy in effect-service-conventions.md to reduce review noise and focus on the diff. The agent now excludes **/*.test.ts and all TSX files, no longer uses Git or GitHub read-only tools, and reports only violations on changed lines instead of repository-wide call-site cleanup. Guidance is consolidated and shortened across Effect imports, service definitions, dependency acquisition, runtime boundaries, and errors; added a requirement to delete obsolete service files and compatibility shims during service moves. A clean review must return exactly All clear.

• Behavioral Change: agent scope and tool set change — removed Git/GitHub tools and TSX/test file inclusion may miss issues previously caught in those files.

📊 Macroscope summarized 80527dd. 1 file reviewed, 1 issue evaluated, 0 issues filtered, 1 comment posted

🗂️ Filtered Issues

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.4 KiB	13.2 KiB	−230 B (−1.7%)	15.1 KiB	✅
Codex	Thread snapshot wire	6.9 KiB	6.9 KiB	0 B (0.0%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.6 KiB	6.3 KiB	−230 B (−3.4%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.0 KiB	55.5 KiB	−1.5 KiB (−2.6%)	66.4 KiB	✅
Codex	Live turn messages	10	8	−2 (−20.0%)	21	✅
Claude	Total thread wire	13.3 KiB	13.3 KiB	+15 B (+0.1%)	15.1 KiB	✅
Claude	Thread snapshot wire	6.9 KiB	6.9 KiB	−4 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	+19 B (+0.3%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	56.4 KiB	56.4 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	10	10	0 (0.0%)	21	✅

Baseline: 194f838 · PR result: 80527dd · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 109.4 KiB
• Claude decoded thread snapshot: 110.1 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.