Skip to content
This repository was archived by the owner on Oct 1, 2026. It is now read-only.

chore: sync fork with upstream tasks-axi - #2

Merged
yjuyjuy merged 6 commits into
mainfrom
fm/sync-tasks-axi-upstream
Aug 13, 2026
Merged

yjuyjuy merged 6 commits into
mainfrom
fm/sync-tasks-axi-upstream

Conversation

@yjuyjuy

@yjuyjuy yjuyjuy commented Aug 13, 2026

Copy link
Copy Markdown
Owner

Summary

Merges the latest upstream (kunchenguid/tasks-axi, upstream/main) into our fork's main via a real merge commit, preserving our two fork commits.

Upstream commits brought in (our main was 5 behind)

Fork commits preserved

Both of our commits from PR #1 remain on the merged lineage:

This is a merge commit onto our fork lineage, not a rebase, so no fork history was rewritten.

Validation

  • pnpm install --frozen-lockfile: clean.
  • pnpm run build (tsc): passed.
  • pnpm test (vitest): 444 passed, 1 skipped (the CI-skipped real-firstmate-backlog fixture check).
  • pnpm run build:skill -- --check: SKILL.md up to date, no drift.

The merge was clean with no conflicts.

kunchenguid and others added 6 commits August 6, 2026 21:02
* 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.
Merge upstream/main (through kunchenguid#41) into fork main. Brings in:
- fix(cli): speed up standalone version queries (kunchenguid#34)
- chore(main): release tasks-axi 0.2.5 (kunchenguid#35)
- fix(cli): accept canonical Forgejo pull request URLs (kunchenguid#36)
- chore: remove committed no-mistakes evidence (kunchenguid#39)
- chore: gitignore no-mistakes evidence dir (kunchenguid#41)

Preserves fork commits: priority-first-class ranking and resumable session token (PR #1).
@yjuyjuy
yjuyjuy merged commit 433e34d into main Aug 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants