Skip to content

ui: fix Vercel build — explicit config, drop public flag, relax pnpm engine constraint - #23743

Merged
suresh-hashicorp merged 1 commit into
mainfrom
suresh/ui/fix-vercel-build
Jul 17, 2026
Merged

ui: fix Vercel build — explicit config, drop public flag, relax pnpm engine constraint#23743
suresh-hashicorp merged 1 commit into
mainfrom
suresh/ui/fix-vercel-build

Conversation

@suresh-hashicorp

@suresh-hashicorp suresh-hashicorp commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Description

Isolates the Vercel build fixes from #23707 into a standalone PR targeting main.

Three files changed, all under ui/:

ui/vercel.json

  • Add $schema for editor/validator support
  • Remove unsupported "public": true flag (caused Vercel deploy errors)
  • Add explicit installCommand: pnpm install --no-frozen-lockfile
    --no-frozen-lockfile avoids install failures when Vercel's Node environment
    produces a different lockfile hash than the one committed
  • Add explicit buildCommand: make dist-vercel
  • Add explicit outputDirectory: dist

ui/package.json

  • Relax engines.pnpm from ">= 10.*" to ">= 9" so Vercel's bundled
    pnpm 9 satisfies the constraint without requiring a global install step

ui/README.md

  • Add one-line note pointing to ./vercel.json for Vercel preview deployment config

Testing & Reproduction steps

Vercel preview deployment on a branch should now complete without:

  • ERR_INVALID_ARG_TYPE / unsupported config key errors (public)
  • ERR_PNPM_FROZEN_LOCKFILE failures
  • pnpm engine version mismatch errors

Links

PR Checklist

  • not a security concern
  • updated test coverage (n/a — config-only change)
  • external facing docs updated (n/a)
  • appropriate backport labels added

…engine constraint

- Add $schema, installCommand (pnpm install --no-frozen-lockfile),
  buildCommand (make dist-vercel), and outputDirectory to ui/vercel.json
- Remove unsupported 'public' flag from ui/vercel.json
- Relax engines.pnpm constraint from '>= 10.*' to '>= 9' in ui/package.json
  so Vercel's bundled pnpm 9 can satisfy it during install
- Document workspace Vercel config location in ui/README.md

Backport of changes from #23707.
@suresh-hashicorp
suresh-hashicorp force-pushed the suresh/ui/fix-vercel-build branch from 3fe020e to c66a1aa Compare July 17, 2026 09:11
@suresh-hashicorp suresh-hashicorp added pr/no-changelog PR does not need a corresponding .changelog entry backport/all Apply backports for all active releases per .release/versions.hcl labels Jul 17, 2026
@suresh-hashicorp
suresh-hashicorp merged commit f69ddcf into main Jul 17, 2026
107 of 119 checks passed
@hc-github-team-consul-core hc-github-team-consul-core added backport/2.0 Changes are backported to 2.0 backport/ent/1.21 changes are backported to 1.21 ent backport/ent/1.22 Changes are backported to 1.22 ent labels Jul 17, 2026
@hc-github-team-consul-core

Copy link
Copy Markdown
Collaborator

📣 Hi @suresh-hashicorp! a backport is missing for this PR [23743] for versions [1.21,1.22,2.0] please perform the backport manually and add the following snippet to your backport PR description:

<details>
	<summary> Overview of commits </summary>
		- <<backport commit 1>>
		- <<backport commit 2>>
		...
</details>

@hc-github-team-consul-core

Copy link
Copy Markdown
Collaborator

📣 Hi @suresh-hashicorp! a backport is missing for this PR [23743] for versions [1.21,1.22] please perform the backport manually and add the following snippet to your backport PR description:

<details>
	<summary> Overview of commits </summary>
		- <<backport commit 1>>
		- <<backport commit 2>>
		...
</details>

2 similar comments
@hc-github-team-consul-core

Copy link
Copy Markdown
Collaborator

📣 Hi @suresh-hashicorp! a backport is missing for this PR [23743] for versions [1.21,1.22] please perform the backport manually and add the following snippet to your backport PR description:

<details>
	<summary> Overview of commits </summary>
		- <<backport commit 1>>
		- <<backport commit 2>>
		...
</details>

@hc-github-team-consul-core

Copy link
Copy Markdown
Collaborator

📣 Hi @suresh-hashicorp! a backport is missing for this PR [23743] for versions [1.21,1.22] please perform the backport manually and add the following snippet to your backport PR description:

<details>
	<summary> Overview of commits </summary>
		- <<backport commit 1>>
		- <<backport commit 2>>
		...
</details>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/all Apply backports for all active releases per .release/versions.hcl backport/ent/1.21 changes are backported to 1.21 ent backport/ent/1.22 Changes are backported to 1.22 ent backport/2.0 Changes are backported to 2.0 pr/no-changelog PR does not need a corresponding .changelog entry theme/ui Anything related to the UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants