docs: add known issues for ag create silent output (#3357) and arg shorthand (#3352)#3390
Conversation
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
❌ 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.
c4eadc1 to
a19eb89
Compare
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 foobarcreates 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.