fix(daemon): restore Antigravity STDIN pipe by omitting legacy -p flag by pratikforge · Pull Request #6060 · nexu-io/open-design

pratikforge · 2026-07-24T15:21:34Z

Why

This fixes the ENAMETOOLONG crash on Windows when using the latest agy 1.1.x versions with large plugins (like the Default design router).

As noted in #5495, agy 1.1.x broke the legacy -p - CLI flag contract, treating the dash as an empty prompt rather than a STDIN sentinel. However, the background STDIN pipe itself still works natively in 1.1.x if the flag is omitted entirely.

This PR removes the explicit -p flag argument from buildArgs. When promptViaStdin is enabled, the daemon now correctly pipes the payload natively, completely resolving the Windows limit crash without needing temporary files or legacy binary downgrades.

What users will see

Windows users will no longer experience spawn failed: spawn ENAMETOOLONG crashes when generating designs with large context windows.
Users can safely use the latest global agy versions (1.1.x) instead of being pinned to 1.0.3.

Proof it works!
Here is Open Design flawlessly rendering a massive payload using the latest agy 1.1.x on Windows without crashing:

Surface area

UI — new page / dialog / panel / menu item / setting / empty state in apps/web or apps/desktop (including Electron menu bar)
Keyboard shortcut — new or changed
CLI / env var — new od subcommand or flag, new tools-dev / tools-pack / tools-pr flag, or new OD_* env var
API / contract — new /api/* endpoint, new SSE event, or changed shape in packages/contracts
Extension point — new entry under skills/, design-systems/, design-templates/, or craft/, or change to the skills protocol
i18n keys — added new translation keys (see TRANSLATIONS.md for the locale workflow)
New top-level dependency — adding any new entry to the root package.json
Default behavior change — changes what existing users experience without opting in
None — internal refactor, docs, tests, or translation update only

Bug fix verification

Test path that reproduces the bug: apps/daemon/tests/runtimes/agent-args.test.ts
Did the test go red on main and green on this branch? yes
We explicitly verified the new STDIN logic against agy 1.1.5 on Windows where a 30KB payload previously triggered ENAMETOOLONG.

Validation

npx vitest run tests/runtimes/agent-args.test.ts
End-to-end testing with a large context generation via npx pnpm run tools-dev.

agy's -p/--print/--prompt takes the prompt as a command-line argument; it does not read from stdin and has no '-' stdin sentinel. The adapter invoked 'agy -p -' with promptViaStdin, so agy received the literal '-' as an empty prompt and replied "your request was empty or a placeholder (-)". - antigravity.ts: emit ['-p', <prompt>] and set promptViaStdin: false - executables.ts / app-config.ts: add missing AGY_BIN override key + allowlist entry so agy can be pointed at an explicit binary when auto-detection fails (parity with every other agent) - agent-args.test.ts: assert the new argv contract

…files

…payload files" This reverts commit 9bfbe3d.

…cal agy environment

lefarcen

Hey @pratikforge — the regression write-up and the Windows reproduction are clear, especially the way you tied the ENAMETOOLONG failure back to the Antigravity prompt-delivery path. Before pool review picks this up, could you add the missing Surface area checklist and a short Validation section describing exactly what you ran to verify the fix?

Related: #5585 and #5997 are also open against this same Antigravity prompt-delivery area, so it would be good to compare approaches and avoid duplicate merge work.

lefarcen · 2026-07-24T15:26:45Z

🧪 Queued for QA validation — this PR changes a live runtime path, so we’ll keep the needs-validation label on it and run the manual QA pass once it is merge-ready. Nothing needed from you right now; we’ll update here after validation. Thanks for the contribution! 🙏

Copilot

Pull request overview

This PR updates the daemon’s Antigravity (agy) runtime integration to avoid the legacy -p - prompt-flag contract and improve compatibility with newer agy behavior (notably on Windows where long argv can trigger ENAMETOOLONG).

Changes:

Adjust Antigravity runtime argument construction to stop relying on the legacy -p - stdin-sentinel behavior.
Add AGY_BIN as a supported env override for resolving the Antigravity executable.
Update unit tests that pin the Antigravity argv/prompt-delivery contract.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File	Description
apps/daemon/src/runtimes/defs/antigravity.ts	Changes Antigravity runtime startup behavior (argv building, PATH handling, prompt delivery assumptions).
apps/daemon/tests/runtimes/agent-args.test.ts	Updates the pinned Antigravity argv/prompt-delivery expectations.
apps/daemon/src/runtimes/executables.ts	Adds `AGY_BIN` as the env var override key for Antigravity binary resolution.
apps/daemon/src/app-config.ts	Allows `AGY_BIN` in the agent CLI env allowlist for Antigravity.

Comments suppressed due to low confidence (2)

apps/daemon/tests/runtimes/agent-args.test.ts:563

The model-selection argv assertion still expects -p <prompt>, but after switching antigravity back to stdin prompt delivery, buildArgs should stay prompt-free and only include --log-file (plus any other non-prompt flags). Update this deep-equality assertion accordingly.

      agentLogFilePath: '/tmp/od-agy-test.log',
      antigravitySettingsPath: join(settingsDir, 'settings.json'),
    });
    assert.equal(withModel.includes('--model'), false);
    assert.deepEqual(withModel, ['--log-file', '/tmp/od-agy-test.log', '-p', 'hi']);

apps/daemon/tests/runtimes/agent-args.test.ts:580

Follow-up/first-turn assertions still expect -p <prompt> argv. With stdin prompt delivery, buildArgs should return the same prompt-free argv regardless of turn (and should still avoid -c). Update these expectations to match the new contract.

  const followUp = antigravity.buildArgs('next message', [], [], {}, {
    hasPriorAssistantTurn: true,
  });
  assert.deepEqual(followUp, ['-p', 'next message']);
  assert.equal(followUp.includes('-c'), false);

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

+import { dirname, join, delimiter } from 'node:path';
+
+const localAgyPath = join(process.cwd(), '.agy');
+if (existsSync(localAgyPath) && !process.env.PATH?.includes(localAgyPath)) {
+  process.env.PATH = localAgyPath + delimiter + process.env.PATH;
+}


  // plain-stream models like Gemini 3.5 Flash.
  buildArgs: (
-    _prompt,
+    prompt,
    _imagePaths,
    _extra = [],


+    // Flag order is load-bearing on agy: `agy -p --log-file /tmp/x <prompt>`
+    // runs, while the reverse leaves the log empty.
    if (runtimeContext.agentLogFilePath) {
      args.push('--log-file', runtimeContext.agentLogFilePath);
    }


+test('antigravity passes the prompt as the -p argument (print mode)', () => {
  assert.equal(antigravity.bin, 'agy');
  assert.equal(antigravity.streamFormat, 'plain');
-  assert.equal(antigravity.promptViaStdin, true);
+  assert.equal(antigravity.promptViaStdin, false);



lefarcen · 2026-07-24T15:34:36Z

Hey @pratikforge — there are a couple of concrete follow-ups in the current review thread now: the test expectations still need to match the stdin-based prompt contract, and the module-level PATH mutation in antigravity.ts needs another pass. Once those are addressed, please also fill in Surface area, Bug fix verification, and Validation in the PR body so the next review round has the full context.

💡 To drive this PR to merge hands-free, paste this to your AI coding agent (Claude Code / Codex / opencode / Cursor …):
Take over nexu-io/open-design#6060 until it merges — read https://raw.githubusercontent.com/nexu-io/looper/main/skills/pr-takeover/SKILL.md and follow it.

lefarcen

Hey @pratikforge — thanks for updating the PR body; the added sections make the intent much easier to follow. There’s still one template pass left before pool review can scope this quickly: could you turn Surface area into the checklist form from the template, and make Bug fix verification / Validation a bit more concrete about the exact red→green repro and command(s) you ran?

pratikforge · 2026-07-24T15:48:28Z

Hey @lefarcen , thanks for the review!

I've just pushed a new commit that addresses all the follow-ups:

Removed the module-level .agy PATH mutation hack.
Updated agent-args.test.ts to align the test assertions with the new prompt-free STDIN contract.
Added the Surface area, Bug fix verification, and Validation sections to the PR body for context.

Everything is green on my end, let me know if anything else is needed for merge!

pratikforge · 2026-07-24T15:51:55Z

All done! Updated the description to match the template checklist and added the exact test path + repro details. @lefarcen

lefarcen · 2026-07-24T15:57:58Z

Thanks @pratikforge — I can see the body updates landed, and the bot-side follow-ups you were responding to are covered now. From here the next step is pool review on the current head; once the PR is merge-ready, QA can pick up the needs-validation pass.

ARareUsername and others added 5 commits July 21, 2026 02:43

fix(daemon): bypass Windows 32kb command line limit via .tmp payload …

9bfbe3d

…files

Revert "fix(daemon): bypass Windows 32kb command line limit via .tmp …

9ad5670

…payload files" This reverts commit 9bfbe3d.

fix(daemon): switch away from legacy -p STDIN behavior and isolate lo…

c9f13d2

…cal agy environment

Merge branch 'feat/isolated-agy-env'

c1221ab

Copilot AI review requested due to automatic review settings July 24, 2026 15:21

Copilot started reviewing on behalf of pratikforge July 24, 2026 15:22 View session

lefarcen requested a review from nettee July 24, 2026 15:25

lefarcen added size/S PR changes 20-100 lines risk/high High risk: apps/desktop, daemon, auth, migration, workflows, package deps type/bugfix Bug fix needs-validation Runtime change detected; needs human or /explore agent validation. labels Jul 24, 2026

lefarcen mentioned this pull request Jul 24, 2026

Antigravity agent adapter fails to pass prompts: agy CLI reads -p - literally instead of from stdin #5495

Open

lefarcen reviewed Jul 24, 2026

View reviewed changes

Copilot AI reviewed Jul 24, 2026

View reviewed changes

pratikforge mentioned this pull request Jul 24, 2026

[Bug]: Antigravity CLI on 0.14.1 #5448

Open

2 tasks

fix(daemon): address review feedback on STDIN tests and PATH mutation

677db29

lefarcen reviewed Jul 24, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix(daemon): restore Antigravity STDIN pipe by omitting legacy -p flag#6060

fix(daemon): restore Antigravity STDIN pipe by omitting legacy -p flag#6060
pratikforge wants to merge 6 commits into
nexu-io:mainfrom
pratikforge:fix-agy-stdin

pratikforge commented Jul 24, 2026 •

edited

Loading

Uh oh!

lefarcen left a comment

Uh oh!

lefarcen commented Jul 24, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

lefarcen commented Jul 24, 2026

Uh oh!

lefarcen left a comment

Uh oh!

pratikforge commented Jul 24, 2026

Uh oh!

pratikforge commented Jul 24, 2026

Uh oh!

lefarcen commented Jul 24, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

pratikforge commented Jul 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Why

What users will see

Surface area

Bug fix verification

Validation

Uh oh!

lefarcen left a comment

Choose a reason for hiding this comment

Uh oh!

lefarcen commented Jul 24, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

lefarcen commented Jul 24, 2026

Uh oh!

lefarcen left a comment

Choose a reason for hiding this comment

Uh oh!

pratikforge commented Jul 24, 2026

Uh oh!

pratikforge commented Jul 24, 2026

Uh oh!

lefarcen commented Jul 24, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

pratikforge commented Jul 24, 2026 •

edited

Loading