Skip to content

docs: add known issues for ag create silent output (#3357) and arg shorthand (#3352)#3390

Merged
aegis-gh-agent[bot] merged 1 commit into
developfrom
docs/fix-3357-3352-cli-known-issues
May 14, 2026
Merged

docs: add known issues for ag create silent output (#3357) and arg shorthand (#3352)#3390
aegis-gh-agent[bot] merged 1 commit into
developfrom
docs/fix-3357-3352-cli-known-issues

Conversation

@OneStepAt4time

Copy link
Copy Markdown
Owner

What

Documents two CLI known issues that need code fixes (Hep's domain):

#3357 — ag create sometimes shows no output

Session is created but CLI doesn't print the ID/curl commands. Users think it failed.

Docs fix: Added known issue callout near the ag create output format + troubleshooting entry.

#3352 — unrecognized args create sessions

ag foobar creates a "foobar" session instead of erroring. The shorthand syntax treats any word as a brief.

Docs fix: Added troubleshooting entry explaining the behavior.

Note

Both need CLI code changes to properly fix. This PR documents the workarounds so users aren't confused in the meantime.

@aegis-gh-agent aegis-gh-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review: Request Changes

This PR has no docs content. The diff only shows src/commands/run.ts changes (streaming output fix from #3368) which were already merged as PR #3389.

No .md files are changed. The known issue callouts for #3357 and #3352 described in the PR body are not in the diff.

This looks like the wrong branch was pushed — the docs changes are missing. Please push the actual docs content (known issue callouts + troubleshooting entries) to this branch.

@aegis-gh-agent aegis-gh-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❌ Changes requested — CI failing.

test (ubuntu-latest, 20) is failing (3m0s timeout):
https://github.com/OneStepAt4time/aegis/actions/runs/25855563229/job/75972400324

Gate 3 (CI green) does not pass. Cannot merge with failing tests.

The code changes look reasonable (update response shape to messages, skip non-text content types, handle crashed status), but we need to understand why the Node 20 test job is timing out. Could be a flaky CI run — please rebase or push an empty commit to re-trigger CI, or investigate the failure.

…orthand (#3352)

- Added known issue callout near ag create output format
- Added troubleshooting entries for both issues
- These are CLI code bugs — docs document the workaround until fixed

References: #3357, #3352
@OneStepAt4time OneStepAt4time force-pushed the docs/fix-3357-3352-cli-known-issues branch from c4eadc1 to a19eb89 Compare May 14, 2026 11:31

@aegis-gh-agent aegis-gh-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.

Clean docs-only PR. Known issue callout for ag create silent output + two troubleshooting entries (silent output + arg shorthand). No code files. CI green.

@aegis-gh-agent aegis-gh-agent Bot merged commit 0b49c1f into develop May 14, 2026
17 checks passed
@aegis-gh-agent aegis-gh-agent Bot deleted the docs/fix-3357-3352-cli-known-issues branch May 14, 2026 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant