feat: support npm release history baselines#14
Merged
Conversation
There was a problem hiding this comment.
1 issue found across 53 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="src/config.ts">
<violation number="1" location="src/config.ts:24">
P2: Scoped package-name validation misses the `:` rejection, so invalid spec-like inputs (for example `@scope/name:latest`) are accepted.</violation>
</file>
Note: This PR contains a large number of files. Free-tier reviews are limited to 40 files per PR, so some files may not have been reviewed. cubic prioritizes the most important files to review. Paid plans can review more files.
You're on the cubic free plan with 9 free PR reviews remaining this month. Upgrade for unlimited reviews.
Re-trigger cubic
…omparisons # Conflicts: # src/index.ts
Bundle Size ReportCompared current build against latest npm release
Historical comparison: latest + 10 previous npm releases
|
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
tarball-uriinput with npmpackage-namerelease baseline resolution.latestrelease plus up to 10 previous stable releases and include historical comparisons in the JSON report.<details>section.main.Testing
pnpm testpnpm run buildpnpm run lintpnpm run typecheckChecklist
dist/index.jswhen runtime source, dependencies, or action metadata changed.Intentional compatibility note: this is a breaking input-contract change from
tarball-uritopackage-name, captured in the OpenSpec proposal.Summary by cubic
Adds npm release baselines and historical comparisons: replace the
tarball-uriinput withpackage-name, compare against the npmlatestrelease and up to 10 previous stable releases, and keep latest totals as the primary outputs. Also updates the repo CI to publishvitesttest reports and coverage.New Features
package-namereplacestarball-uri(breaking).latest+ up to 10 previous stable releases, and compare gzip sizes for configured files.packageName,baseline.version, andhistory[]; PR comments include a collapsed historical section.pnpm run test:ciusingvitest+@vitest/coverage-v8, publish test report viadorny/test-reporter, and add a coverage summary.Migration
package-nameand keepfilesas-is.packageName,baseline.version,history) while existing totals remain unchanged.checks: writefor the test reporter; action consumers do not need this.Written for commit 0f5a252. Summary will update on new commits. Review in cubic