chore: sync upstream main (10 commits) - #5
Closed
yjuyjuy wants to merge 11 commits into
Closed
Conversation
* perf(cli): answer --version through the axi-sdk-js fast path Extract the package-version helper out of the heavy `src/cli.ts` graph into a leaf `src/version.ts` (node builtins only), and rewrite `bin/tasks-axi.ts` to answer a bare `-v`/`-V`/`--version` via `axi-sdk-js/fast-path`, dynamically importing the command graph only for everything else. Bumps axi-sdk-js to ^0.1.10 for the `./fast-path` subpath export. Version output is byte-identical and all other argv shapes still route through `runAxiCli` unchanged. Guarded by a deterministic ESM loader module trace with a negative control plus flag parity; no wall-clock assertion in CI. * no-mistakes(document): Confirm fast-path docs and lint cleanliness * no-mistakes: apply CI fixes
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
* fix: accept canonical Forgejo pull request URLs as typed PR links One classification seam (isPrUrl in src/pr-url.ts) now decides what counts as a PR URL for prose link derivation, done/add --pr validation, and public-followup pr_url deliverables: canonical GitHub https://github.com/<owner>/<repo>/pull/<n> or Forgejo https://<host>/<owner>/<repo>/pulls/<n> with a positive, no-leading-zero number. Near misses (issue routes, singular/plural route confusion, trailing slash, query/fragment, whitespace, userinfo, ports, encoded separators, malformed segments) are rejected as --pr / pr_url values and derive as doc links, never pr. Fixes kunchenguid#19 * fix: validate pr links against the untrimmed input Review follow-ups from pipeline run 01KZFA73D662DCSJ4HVWKW21QX: --pr values and pr-kind addLinks are validated before any trim, so whitespace-padded input is rejected instead of normalized. The literal NUL byte in test/pr-url.test.ts is now written as a unicode source escape so git treats the file as text; an embedded-space rejection case is added alongside it.
…nchenguid#42) Co-authored-by: Kun Chen <kun-1@kunchenguid.com>
…henguid#47) Replace the inline gate `run:` block in .github/workflows/no-mistakes-required.yml with a thin caller of kunchenguid/no-mistakes/.github/actions/require-no-mistakes, pinned to an immutable commit SHA. Enforcement logic and its tests now live upstream, so this repository no longer carries a hand-copied script that can drift from its siblings. Drop `synchronize` from the pull_request trigger: the verdict is a pure function of the PR body, and the pipeline pushes before it writes the Pipeline section, so a push-triggered run pinned a failure to a head whose body the same run was about to fix. This repo's ruleset is advisory with no required status check, so dropping the trigger cannot wedge a merge. Remove test/workflows/no-mistakes-gate.test.ts, which extracted and executed the now-absent inline block, and point AGENTS.md at the shared action.
) * fix(skill): shrink SKILL.md to a CLI-deferring stub Installed skills go stale when the npm package is bumped. Keep only identity frontmatter plus pointers to live CLI help so regeneration cannot re-inflate baked command docs. Co-authored-by: Cursor <cursoragent@cursor.com> * no-mistakes(review): Align skill documentation with minimal generator contract * no-mistakes(document): Consolidate generated skill documentation --------- Co-authored-by: Cursor <cursoragent@cursor.com>
Owner
Author
|
Superseded by #6 (landed through the no-mistakes pipeline gate). |
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
3 participants
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.
Merges 10 upstream commits from kunchenguid/tasks-axi into fork main.
Upstream commits
d9175b6fix: make the generated skill defer to live CLI guidance (fix: make the generated skill defer to live CLI guidance kunchenguid/tasks-axi#48)4c09455ci: migrate the no-mistakes gate to the shared composite action (ci: migrate no-mistakes gate to shared action kunchenguid/tasks-axi#47)24ddd22ci: bind the no-mistakes attestation to the current PR head (ci: bind the no-mistakes attestation to the current PR head kunchenguid/tasks-axi#46)7b9291fci: require no-mistakes pipeline attestation in the gate (ci: require no-mistakes pipeline attestation in the gate kunchenguid/tasks-axi#45)48a5a90chore(agents): use @AGENTS.md import instead of CLAUDE.md symlink (chore(agents): use @AGENTS.md import instead of CLAUDE.md symlink kunchenguid/tasks-axi#42)cb97c32chore: gitignore no-mistakes evidence dir (chore: gitignore no-mistakes evidence dir (contributor safety) kunchenguid/tasks-axi#41)8acfbebchore: remove committed no-mistakes evidence (chore: remove committed no-mistakes evidence (now on orphan branch) kunchenguid/tasks-axi#39)9a86c7cfix(cli): accept canonical Forgejo pull request URLs (fix(cli): accept canonical Forgejo pull request URLs kunchenguid/tasks-axi#36)44a41f3chore(main): release tasks-axi 0.2.5 (chore(main): release tasks-axi 0.2.5 kunchenguid/tasks-axi#35)92911b2fix(cli): speed up standalone version queries (fix(cli): speed up standalone version queries kunchenguid/tasks-axi#34)Test result
444 tests passed, 1 skipped (expected). Build clean. Zero merge conflicts.