Repository navigation
Rename to @versionone/agility-sdk; publish workflow; attachment fixes; C4 - #47
Merged
jallers merged 7 commits intoNov 7, 2026
Merged
Conversation
From gh-v1's C4:
- attachments.get reports the Content-Type the server sent the bytes
with (falling back to the Attachment's recorded type, which stays as
recordedContentType). A request can now take onResponse to see a
successful response's status and headers; wire/attachment's getFile
uses it, and also reads the filename from Content-Disposition.
- attachments.page returns { attachments, total, more }. It rests on a new
assetApi.queryPage, which sends needTotal and reads the total api/asset
reports in resultMetaData (query.v1 ignores needTotal). list is unchanged.
Live (v1sdktesting): text/plain served, total 2 with more after a page of 1.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016NSWj9gL1oCubj5e2anVkb
Track C is done: gh-v1 talks to Agility only through the vendored SDK, apart from the gh v1 api passthrough. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016NSWj9gL1oCubj5e2anVkb
Before the first publish. The @versionone scope already has another SDK (ctm-pipeline-builder-sdk), and the product is Agility. package.json gains a description, keywords, repository, homepage and bugs for npm; the README leads with what the package is, how to install the alpha, and that it is not a compatible successor to v1sdk 2.x. Plans, handoffs and the CommonJS install test use the new name. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016NSWj9gL1oCubj5e2anVkb
jallers
changed the title
.github/workflows/publish.yml runs on a v* tag: checks the tag equals
package.json's version, runs verify, takes the dist-tag from the
prerelease id (alpha, beta, rc; latest only for a plain version), and
publishes with provenance through npm trusted publishing (NPM_TOKEN as a
fallback for the first publish). package.json: 1.0.0-alpha.0 and
publishConfig { access: public, provenance: true }. README: Releasing.
Not published yet: the license is undecided (master's LICENSE.md is BSD
3-clause; package.json says MIT with no license file).
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016NSWj9gL1oCubj5e2anVkb
jallers
changed the title
Jim chose MIT for @versionone/agility-sdk. LICENSE holds the text (npm packs it automatically); package.json already says MIT. The 2.x code's BSD 3-clause LICENSE.md on master is unaffected. Plan 001 records the decision; A6 has only the first publish and the npmjs link left. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016NSWj9gL1oCubj5e2anVkb
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016NSWj9gL1oCubj5e2anVkb
tsup reads entries as glob patterns, where a backslash is an escape, so on
Windows the CJS test's absolute entry path (D:\...\src\index.ts) matched
nothing ("Cannot find ...\src\index.ts", reported by Jim). It now hands
tsup a forward-slash path.
CI adds a Windows / Node 24 job so this kind of break shows up in a PR,
and the tarball upload uses the renamed package's file name.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016NSWj9gL1oCubj5e2anVkb
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.
This renames the package to
@versionone/agility-sdk, adds the npm publish workflow (A6) and an MIT license, fixes two attachment gaps from gh-v1's C4, and ticks C4.@versiononescope already hasctm-pipeline-builder-sdk, and the product is Agility.package.jsonnow has a description, keywords and links. The README leads with how to install the alpha and states that this isn't a compatible successor tov1sdk2.x.v*tag runs.github/workflows/publish.yml. It checks the tag matchespackage.json, runsverify, and publishes with provenance through npm trusted publishing. A prerelease goes out on its own dist-tag (alpha,betaorrc), solatestis never a prerelease. The version is now1.0.0-alpha.0. The license is MIT (LICENSE, included in the package).master's BSD 3-clauseLICENSE.mdstays with the 2.x code.attachments.getreports the Content-Type the server sent, and a newattachments.pagereturns{ attachments, total, more }. Plan 001 ticks C4, which completes Track C, and records the license decision.After merge, the first version is published once by hand. Then the package is linked to
publish.ymlon npmjs, and later releases publish from tags (see the README's "Releasing" section).All 127 unit tests pass.
npm packholds onlydist/, the README,LICENSEandpackage.json.🤖 Generated with Claude Code
https://claude.ai/code/session_016NSWj9gL1oCubj5e2anVkb