Skip to content

For alpha 20#54

Merged
baxyz merged 12 commits intomainfrom
for-alpha-20
Apr 25, 2026
Merged

For alpha 20#54
baxyz merged 12 commits intomainfrom
for-alpha-20

Conversation

@baxyz
Copy link
Copy Markdown
Contributor

@baxyz baxyz commented Apr 24, 2026

Summary

This PR covers several independent improvements targeting the v2.0.0-alpha.20 release.

CI / Release workflow

  • Replace app-id with client-id for actions/create-github-app-token steps (API change)
  • Extract website documentation refresh into a dedicated trigger-website-docs job (runs after publish, isolated failure scope)

New helpers

  • version/isPrerelease — detect whether a version string or ParsedVersion is a prerelease; handles build metadata (1.0.0+build-123false)
  • version/stringify — serialize a ParsedVersion back to a string
  • number/formatSize — human-readable byte formatting (B / KB / MB / GB)
  • url/parsePackageRepository — parse the repository field of package.json into a structured { host, slug, owner, repo, gistId, directory } object; supports all npm-specified formats (shorthand, SSH, HTTPS, gist)

Script improvements

  • scripts/version/inject-since.ts — align CLI entry-point guard with endsWith pattern used elsewhere; fix false-positive on 1.0.0+build-123 in isPrerelease
  • scripts/build/build-website-metadata.ts — companion-type detection now includes interface kinds (fixes ParsedVersion incorrectly getting a standalone helper page); interface typeDefinition rendering now filters by ReflectionKind.Property and handles method signatures

Tests & quality

  • 100% coverage maintained (2868 tests)
  • Property-based tests with fast-check for formatSize, isPrerelease, parsePackageRepository
  • Dead code removed from parsePackageRepository (guards made impossible by regex structure)

- replace app-id with client-id for GitHub app token
- add dedicated job for triggering website documentation refresh
Copilot AI review requested due to automatic review settings April 24, 2026 20:12
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the release workflow to align GitHub App token generation with client-id and to move the website documentation refresh trigger into a dedicated post-publish job.

Changes:

  • Switched actions/create-github-app-token@v3 input from app-id to client-id for existing app-token steps.
  • Removed the inline docs-trigger steps from the publish job and replaced them with a separate trigger-website-docs job that runs after publish with its own concurrency key.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/release.yml
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 24, 2026

✅ PR Validation Passed

All checks passed and coverage target reached!


📋 Pipeline Status

Job Status
🔢 Version passing
🏗️ Build passing
🧪 Tests passing
📝 Lint passing
📘 TypeCheck passing
🔐 Security Audit passing
🧾 Conventional Commits passing
🔗 Coherency passing

📊 Code Coverage

Overall Coverage: 100.0% — Target reached! 🎯

Metric Progress Coverage
Lines ▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓ 100%
Branches ▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓ 100%
Functions ▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓ 100%
Statements ▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓ 100%

🧬 Mutation Testing

🟢 Mutation Score: 92.39% — excellent

Metric Progress Score
🟢 Mutation Score ▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓░░ 92.39%

🧬 Mutation testing is informational only and does not block the PR

📊 View full mutation report on Stryker Dashboard


🌐 Runtime Compatibility

Runtime Compatibility: 3/3 — All runtimes passed!

Runtime Status
🟩 Node.js 24 (LTS) v24.14.1
🦕 Deno 2.7.13
🍞 Bun 1.3.13

🔄 Runtime compatibility is informational only and does not block the PR


⏱️ Benchmarks

Benchmarks: 0 suites — completed

⏱️ Benchmarks are informational only and do not block the PR


ℹ️ About this report
  • 🎯 Coverage Target: 100% for all metrics
  • 🧬 Mutation Testing: powered by Stryker (non-blocking)
  • ⏱️ Benchmarks: powered by Vitest Bench (non-blocking)
  • 🔄 This comment updates automatically with each push
  • 📈 Coverage is measured using Vitest + v8

🤖 Generated by @helpers4 CI • 2026-04-25

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 20 out of 20 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread helpers/number/formatSize.spec.ts Outdated
Comment thread helpers/url/parsePackageRepository.ts
Comment thread helpers/version/isPrerelease.ts
Comment thread scripts/version/inject-since.ts Outdated
baxyz added 5 commits April 24, 2026 21:56
- add step to inject version in release workflow
- update CLI entry point for version injection script
- implement parsePackageRepository function
- add tests for various input formats
- create example usage for documentation
- implement isPrerelease function to check version suffix
- add examples for isPrerelease usage
- create tests for string and ParsedVersion inputs
- implement stringify function for semantic versioning
- add examples for stable and prerelease versions
- include tests for stringify functionality
- build interface definition string for interfaces
- update processMember to handle interface kind
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 21 out of 21 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread helpers/url/parsePackageRepository.ts
Comment thread scripts/build/build-website-metadata.ts Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 21 out of 21 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread scripts/version/inject-since.ts
Comment thread .github/workflows/release.yml Outdated
Comment thread .github/workflows/release.yml Outdated
baxyz added 2 commits April 25, 2026 17:42
…eases

- inject @SInCE version for stable releases
- upload modified helpers for stable releases
- download modified helpers for stable releases
@baxyz baxyz merged commit 00f2765 into main Apr 25, 2026
15 checks passed
@baxyz baxyz deleted the for-alpha-20 branch April 25, 2026 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants