Repository navigation
Add NCC-13 software package release profile - #2
Merged
Merged
Conversation
Implement Stage 11 of the NCC integration plan: package-manager semantics over an existing NIP-51 Release Artifact Set (kind 30063) rather than a new event, per NCC-13's own design principle. Adds the ncc13 module (version/version_scheme/channel extraction with the opaque/stable defaults, two-column requires/optional/conflicts, source/commit provenance, the ncc:13:publish operator_for claim, a SemVer-precedence comparator, single-constraint satisfaction, and os/arch artefact matching), a minimal ArtefactMetadata declaration for NIP-94's kind 1063 carrying only NCC-13's own selectors, conformance fixtures and wire vector, a worked release-monitor example, and the spec/matrix/plan documentation. Bumps the NCC repository pin from adad778 to 186c3ad (upstream added ncc-13; purely additive, no earlier convention's text changed) across every existing ncc0X/ncc1X module and vector reference. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01D95PWA2fdNfg7nUvUHBeMi
CI's clippy (1.98) flags `&[value.clone()]` where `std::slice::from_ref` avoids the clone; the local toolchain here (1.94) predates that lint. Use std::slice::from_ref for the two single-element call sites the NCC-13 test added instead of cloning. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01D95PWA2fdNfg7nUvUHBeMi
Owner
Author
|
CI's Generated by Claude Code |
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.
Summary
NCC-PLAN.md: NCC-13 (Software Package Release Profile). NCC-13 defines no new event — it profiles the existing NIP-51 Release Artifact Set (kind 30063) with package-manager metadata, so this adds anncc13module rather than redefining anything NIP-51/94/34 already own.modules/std/ncc13/0.1.0.nsm: theReleaseArtifactSetevent (kind 30063:d,a,version,version_scheme,channel,source,commit, repeatablerequires/optional/conflicts,e,operator_for) and a minimalArtefactMetadataevent for NIP-94's kind 1063 carrying only NCC-13's ownos/arch/formatselectors.opaque/stabledefaults, the two-columnrequires/optional/conflictsbuilder/readers, thencc:13:publishoperator_forclaim (mirroring NCC-10's shape), a SemVer-§11 precedence comparator (prerelease-before-release, prerelease identifier comparison, build metadata ignored), single-constraint satisfaction (=/>/>=/</<=, relational operators scoped tosemveronly per §8.2/§8.3), andos/archartefact matching with theanywildcard..nsfixtures, a pinned wire vector, aconformance/MATRIX.mdrow, and a workedexamples/ncc13-release-monitor.nsbot (publish → upgrade-eligibility judgment → artefact selection), all exercised by new CLI tests plus a runtime unit test covering SemVer precedence and constraint satisfaction.nostr-community-conventionspin fromadad778to186c3ad(upstream addedncc-13; purely additive) across every existingncc0X/ncc1Xmodule and vector reference, and updatesNCC-PLAN.md/ROADMAP.md/spec/README.md/spec/nostr.mdaccordingly.Test plan
cargo build --workspace— cleancargo test --workspace— all tests pass (249 passed, 1 ignored)cargo fmt --check— cleancargo clippy --workspace -- -D warnings— cleanexamples/ncc13-release-monitor.ns's header comment (nscript runwith each event) and confirmed the printed output🤖 Generated with Claude Code
https://claude.ai/code/session_01D95PWA2fdNfg7nUvUHBeMi
Generated by Claude Code