Repository navigation
Close AXI admission gaps (principles 2-7, 9, 10); v0.3.0 - #10
Merged
Merged
Conversation
…g, counts, --interactive opt-in, session ids
…ds, destination-checked no-ops, multi-list paging, kept sibling content, carried selectors in refusals
…t-checked delete no-ops, list next-step fallback, help for own commands
…ields, more selector forms, empty-list next steps, stream diagnostics to stderr
…nt, scope quoting, nested plain-list counts, setup help defaults, cleaner HEY errors
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The axi.md admission review of hey-axi@0.2.0 (pinned to 2af99c1, kunchenguid/axi#226) admitted it only as an exception. It found AXI principles 2-7, 9 and 10 partly unmet. This PR fixes each gap and releases the result as 0.3.0.
These rules stay as they were:
--allow-send. compose and reply still become drafts, and forward, draft send and bulk-reply send are still refused.hey.What changed, by principle
--fullhint. Long plain-text output is cut with a size and a--fullhint, and--fullreturns it whole.count. Lists without HEY's envelope, lists of plain values and results with several lists are counted too. HEY'stotal_count/next_pageare read frommetaor from next to the list (box listings). hey-axi only says "total" when HEY does; otherwise the count reads "N shown; no more pages reported".--quietkeeps the count, the empty state and hey-axi's hints. No data prints "no data returned".--draft=false), unknown letters in stacked shorthands (-vZ), and missing or extra positional arguments all exit 2 before HEY runs.hey commands), with "did you mean" suggestions.kind.meta.ok: true, noop: true(exit 0) only when HEY's failure matches that command's own end-state pattern. Deletes of missing items are no-ops too. Reads, sends and generic conflicts stay errors.tui,mcp, the setup wizard and browser login are refused unless the new--interactiveopt-in is given.HEY_NONINTERACTIVE=1andEDITOR/VISUAL=false.--message -reads it from stdin.hey-axi setup scopewrites.hey-axi.json, so the home view is directory-scoped (box, label or search, plus account).setup hooksalso installs session-end hooks for Claude Code, Codex and OpenCode. These record command names and numeric ids only, tagged with the agent session id when available, and the next home view shows a one-linelast_session.npx -y hey-axi.--account/--base-urlare carried into object and string breadcrumbs, notices and error fix-it lines. The home view says how to see the rest of a truncated list.--helplists its arguments, every flag with its type and default, every global flag with its default, and 2-3 examples.binis an absolute path, with~for the home directory.Also changed:
set-aside,set-aside groupandskillare now runnable.hey --json.Tests
npm test: 141/141 pass. All tests use the fakehey, and the newtest/admission.test.jscovers each item above.npm run skill:check: up to date.Not merged and not published. The npm release (0.3.0) and the axi.md re-pin wait for owner approval.
Independent re-review status
I ran the same codex-based admission review the axi pipeline uses against each branch head. The latest pass (at b21f95a) returned inconclusive; the pass before it (at c5dd79d) returned exception. Every original gap is now resolved or narrowed. The remaining findings are listed below.
Design choices to confirm:
.hey-axi.jsonis set.--interactivestays as an explicit opt-in for person-only commands.Wrapper edge cases still open:
seencan be acknowledged when only one id was already seen.--pageis appended rather than replaced.Upstream axi-sdk-js: the start hook doesn't quote paths, and its PATH lookup can pick a shadowed executable.