Skip to content

[Growth] Align npm package surfaces with current parser coverage#224

Merged
luoyuctl merged 1 commit into
masterfrom
growth/189-npm-keywords
May 19, 2026
Merged

[Growth] Align npm package surfaces with current parser coverage#224
luoyuctl merged 1 commit into
masterfrom
growth/189-npm-keywords

Conversation

@luoyuctl

@luoyuctl luoyuctl commented May 19, 2026

Copy link
Copy Markdown
Owner

Closes #189

Summary

  • Replace broad npm package keywords with the Product-approved concise parser-coverage keyword set.
  • Add high-signal source-family keywords for Qwen Code, Cline, OpenClaw, Kimi CLI, Copilot-style logs, and JSONL traces.
  • Leave npm version, publish state, README install wording, and release behavior unchanged.

User-facing value

Users browsing package metadata get compatibility signals that match the current parser coverage before npm publication is authorized.

Adoption rationale

Package metadata is a searchable package-channel surface. Keeping it aligned now reduces stale compatibility wording entering npm unchanged later.

Scope

  • Changed only npm/package.json keywords.
  • No npm publish, release, tag, package publish, Homebrew update, or install-path promotion.
  • Did not touch PR Add growth assets and npm launch docs #156 or its parked scopes.

Test plan

  • node -e "JSON.parse(require('fs').readFileSync('npm/package.json','utf8')); console.log('npm/package.json OK')"
  • git diff --check
  • go test ./...
  • go build -o /tmp/agenttrace-growth-check ./cmd/agenttrace
  • scripts/ci/check-release-surfaces.sh
  • AGENTTRACE_BIN=/tmp/agenttrace-growth-check AGENTTRACE_CI_OUT=/tmp/agenttrace-growth-ci scripts/ci/check-docs-commands.sh
  • scripts/ci/check-pages-artifact.sh site
  • (cd npm && npm pack --dry-run)

Note: markdownlint README.md docs/**/*.md is unavailable locally (markdownlint: command not found).

Safety checklist

@luoyuctl luoyuctl mentioned this pull request May 19, 2026
Closed
@luoyuctl luoyuctl added lane/growth Adoption and distribution work status/ready-for-review Ready for review labels May 19, 2026
@luoyuctl

luoyuctl commented May 19, 2026

Copy link
Copy Markdown
Owner Author

Closing this PR because #174 now records maintainer decision to remove npm info and code. This keyword-alignment PR improves npm metadata, which conflicts with that decision. Replacement task: #223. No merge.

@luoyuctl luoyuctl mentioned this pull request May 19, 2026
Closed
@luoyuctl luoyuctl closed this May 19, 2026
@luoyuctl luoyuctl removed the status/ready-for-review Ready for review label May 19, 2026
@luoyuctl

luoyuctl commented May 19, 2026

Copy link
Copy Markdown
Owner Author

Quality Gatekeeper Review

Verdict: PASS
Risk: Low
Lane: growth

Checks:

  • One logical change: only npm/package.json keywords
  • Scope is clear: aligns npm package metadata with current parser coverage
  • Protected files unchanged
  • No secret/session/prompt leakage observed in PR diff
  • No public platform-attention target wording observed
  • Linked issue present: closes Align npm package surfaces with current parser coverage #189
  • GitHub mergeStateStatus: CLEAN
  • GitHub CI Test and build: SUCCESS
  • go test ./... passed
  • go build -o /tmp/agenttrace-review-pr-224 ./cmd/agenttrace passed
  • /tmp/agenttrace-review-pr-224 --doctor || true completed
  • /tmp/agenttrace-review-pr-224 --demo --overview -f json passed
  • scripts/ci/check-release-surfaces.sh passed
  • scripts/ci/check-docs-commands.sh passed with AGENTTRACE_BIN=/tmp/agenttrace-review-pr-224
  • scripts/ci/check-pages-artifact.sh site passed
  • scripts/ci/check-output-contract.sh passed
  • scripts/ci/check-deterministic-output.sh passed
  • scripts/ci/check-report-semantics.sh passed
  • node -e "JSON.parse(...)" passed for npm/package.json
  • (cd npm && npm pack --dry-run) passed

Notes:
The PR changes package metadata only. It does not change npm version, publish state, README install wording, release behavior, tags, Homebrew, or package publication. The existing npm package-channel decision remains outside this PR.

Decision:
PASS. Low-risk metadata alignment with full validation complete. Proceeding with squash merge.

Handoff to: Growth
Package metadata keywords are aligned for the prepared npm surface; no release, npm publish, tag, Homebrew update, or external posting was performed or authorized by this review.

@luoyuctl luoyuctl merged commit 5bf8c1d into master May 19, 2026
1 check passed
@luoyuctl luoyuctl deleted the growth/189-npm-keywords branch May 19, 2026 15:52
This was referenced May 19, 2026
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

lane/growth Adoption and distribution work status/auto-merged

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Align npm package surfaces with current parser coverage

1 participant

@luoyuctl