Skip to content

[WIP] Investigate recurring Smoke Antigravity failures#44716

Closed
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/deep-report-investigate-smoke-antigravity-issues
Closed

[WIP] Investigate recurring Smoke Antigravity failures#44716
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/deep-report-investigate-smoke-antigravity-issues

Conversation

Copilot AI commented Jul 10, 2026

Copy link
Copy Markdown
Contributor
  • Investigated root cause: isTruthy() doesn't handle single-quoted equality ('value'), so both {{#if experiments.sub_agent_strategy == 'single_agent'}} and {{#if experiments.sub_agent_strategy == 'sub_agents'}} blocks are always truthy, causing the agent to receive contradictory instructions and sometimes produce no safe outputs
  • Fix isTruthy to support single-quoted strings (backreference \2 to match opening/closing quote)
  • Add tests for single-quoted equality to is_truthy.test.cjs
  • Fix smoke-antigravity.md to use double quotes in {{#if}} conditions
  • Add noop fallback instruction to the Output section
  • Recompile smoke-antigravity.lock.yml

@github-actions

Copy link
Copy Markdown
Contributor

🤖 PR Triage

Field Value
Category chore
Risk 🟢 Low
Score 18/100 (impact:5 + urgency:8 + quality:5)
Action defer

WIP / 0 files changed. Investigation PR with no code changes. Defer until work is substantive.

Generated by 🔧 PR Triage Agent · 206.8 AIC · ⌖ 8.74 AIC · ⊞ 5.4K ·

@github-actions

Copy link
Copy Markdown
Contributor

Hey @copilot-swe-agent 👋 — thanks for digging into the Smoke Antigravity failures! The root cause analysis in the description is well-structured and the planned fixes — patching isTruthy() for single-quoted equality, updating smoke-antigravity.md, and recompiling the lock file — are exactly the right scope.

A couple of things to address before this is ready for review:

  • Empty diff — the PR currently has 0 files changed. The body describes the intended changes but none have been committed yet. The implementation needs to land before this can be reviewed.
  • Missing tests — the plan mentions adding tests to is_truthy.test.cjs, which is great; make sure those actually appear in the diff when the implementation lands.

When you're ready to push the implementation, here's a prompt you can use:

Implement the fixes described in PR #44716 for the Smoke Antigravity failures:

1. In the isTruthy() function, add support for single-quoted string equality comparisons.
2. Add unit tests in `is_truthy.test.cjs` covering single-quoted equality (truthy, falsy, and mixed quote edge cases).
3. Update `smoke-antigravity.md` to use double quotes in all `{{#if}}` conditions.
4. Add a noop fallback instruction to the Output section of `smoke-antigravity.md`.
5. Recompile `smoke-antigravity.lock.yml`.

Generated by ✅ Contribution Check · 103 AIC · ⌖ 9.85 AIC · ⊞ 6.2K ·

@pelikhan pelikhan closed this Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[deep-report] Investigate recurring "Smoke Antigravity produced no safe outputs" failures

2 participants