Skip to content

Repository navigation

fix(codex): keep managed runtime constants internal

MacroscopeApp / Macroscope - Effect Service Conventions completed Sep 29, 2026 in 4m 44s

Effect Service Conventions: 3 violations

  • apps/server/src/provider/CodexInstallation.ts: Service shape is declared outside the Context.Service tag.
  • apps/server/src/provider/CodexAuthCallback.ts: Callback errors expose underlying error messages.
  • apps/server/src/provider/CodexManagedRuntime.ts: Runtime errors expose underlying filesystem error messages.

Details

Warning

  • Invalid effort level "max" in front matter. Fell back to "low". Valid values are: low, medium, high.

Note

Your check run agent prompt is: .macroscope/check-run-agents/effect-service-conventions.md
More information about how Check Run Agents work can be found in our Help Center.

Reviewed all 53 changed files (no prior completed review)

Posted three inline review comments with the expected fixes.

🔍 Agent activity — 4 tool calls

💭 Identifying errors clearly I need to pinpoint clear violations in the code. There seems to be a standalone CodexInstallationService interface that stands out, along with a static layer inline that could be problematic. I suspect the error message in CodexAuthCallback failure and the cause.message in CodexManagedRuntime are connected. It's essential to focus on the strongest indicators and inspect the exact lines in the code viewer to better understand the issues.

1. browse_code — apps/server/src/provider/CodexInstallation.ts

2. browse_code — apps/server/src/provider/CodexAuthCallback.ts

3. browse_code — apps/server/src/provider/CodexManagedRuntime.ts

💭 Summarizing findings on service namespace imports I need to report three key findings regarding service namespace imports, specifically with the numerous named classes like CodexInstallation and ServerSecretStore. There’s an explicit naming convention, especially in two main files: CodexManagedRuntime and ServerConfig. I want to avoid overloading the report but ensure those three findings are strong. Additionally, I'll suggest an inline fix for the cause.message and note the need for a cross-file schema to address callback errors without any differences.

4. modify_pr — Submitted review with 3 inline comments

Agent Credits: 44 credits

Token usage: 9 uncached input · 277,706 cache-read input · 139,980 cache-write input · 753 output
Agent Credits may also include non-token charges from external tools such as web research.

View more details on MacroscopeApp