Skip to content

installer: add explicit asset conflict override - #27

Merged
Kzoeps merged 3 commits into
mainfrom
installer/add-override
Oct 6, 2026
Merged

Kzoeps merged 3 commits into
mainfrom
installer/add-override

Conversation

@Kzoeps

@Kzoeps Kzoeps commented Oct 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add pnpm install:api --override to replace conflicting declared assets with the incoming bundle.
  • Preserve default conflict refusal, full preflight, unchanged-asset skipping, resolver requirements, and existing partial-failure reporting.
  • Replacement is complete: omitted optional configuration fields are cleared, not preserved.
  • Add offline regression tests, CLI help, operator documentation, and a patch Changeset.

Validation

  • pnpm --filter @hypercerts-org/hypercerts-api check passed (generated freshness, lint, types, 280 unit tests).
  • node --test api/tests/unit/tooling/installer-core.test.js api/tests/unit/tooling/installer-cli.test.js api/tests/unit/tooling/profile-installer.test.js passed: 36 tests, independently rerun.
  • git diff --check passed.
  • Changeset validated using an existing lockfile-matching sibling CLI.
  • Root pnpm check blocked by missing docs dependency @vitejs/plugin-react.
  • Independent review completed; clearing omitted script descriptions is the explicitly approved full-replacement behavior.

Not run

  • HTTP runtime tests and live installation; no HappyView uploads performed.

@vercel

vercel Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
api-endpoints Ready Ready Preview Oct 6, 2026 3:32pm UTC

Request Review

@changeset-bot

changeset-bot Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: e80f06b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@hypercerts-org/hypercerts-api Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coderabbitai

coderabbitai Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 9 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 48e2d83d-3834-4372-b85c-bd50f4d4f25e
📥 Commits

Reviewing files that changed from the base of the PR and between 9ebe7cb and e80f06b.

📒 Files selected for processing (7)
  • .changeset/installer-asset-override.md
  • CONTRIBUTING.md
  • api/README.md
  • api/tests/unit/tooling/installer-cli.test.js
  • api/tests/unit/tooling/installer-core.test.js
  • api/tests/unit/tooling/profile-installer.test.js
  • api/tooling/installer.js
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

sonarqubecloud Bot commented Oct 6, 2026

Copy link
Copy Markdown

❌ The last analysis has failed.

See analysis details on SonarQube Cloud

@Kzoeps
Kzoeps merged commit f9670cb into main Oct 6, 2026
7 of 8 checks passed
@Kzoeps
Kzoeps deleted the installer/add-override branch October 6, 2026 15:36
@github-actions github-actions Bot mentioned this pull request Oct 6, 2026

This branch was successfully deployed

1 active deployment
Preview — e80f06b2 Deployed Oct 6, 2026 by vercel[bot]
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.

1 participant