Skip to content

feat: resolve Vite+ executable paths from VpDirs - #131

Merged
fengmk2 merged 13 commits into
mainfrom
agent/xdg-directory-129
Aug 20, 2026
Merged

feat: resolve Vite+ executable paths from VpDirs#131
fengmk2 merged 13 commits into
mainfrom
agent/xdg-directory-129

Conversation

@fengmk2

@fengmk2 fengmk2 commented Aug 19, 2026

Copy link
Copy Markdown
Member

The action reads Vite+'s executable directory from the installed payload. Vite+ 0.3.0 prerelease builds and later versions use VP_DUMP_DIRS=1.

Preview builds also use this check. Versions before the 0.3 release line use the original installer command and the legacy executable directory.

GitHub Actions, Azure Pipelines, and GitLab use the same version check.

Closes #129.

@fengmk2 fengmk2 self-assigned this Aug 19, 2026
@fengmk2
fengmk2 force-pushed the agent/xdg-directory-129 branch from 1134178 to c31c31c Compare August 19, 2026 03:39
Comment thread gitlab/bootstrap.sh Outdated
@fengmk2

fengmk2 commented Aug 19, 2026

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 703e678316

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread gitlab/bootstrap.sh Outdated
@fengmk2

This comment was marked as resolved.

@fengmk2

This comment was marked as resolved.

@fengmk2

This comment was marked as resolved.

@fengmk2
fengmk2 force-pushed the agent/xdg-directory-129 branch from c06cc0b to 511055c Compare August 20, 2026 12:09
@fengmk2
fengmk2 marked this pull request as ready for review August 20, 2026 12:12
Copilot AI lite review requested due to automatic review settings August 20, 2026 12:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings August 20, 2026 12:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@fengmk2

fengmk2 commented Aug 20, 2026

Copy link
Copy Markdown
Member Author

Retested the rebased head 22137151. All GitLab verification jobs pass:

  • basic install, vp pack, and vp test
  • multi-entry install and the second-run 100% cache check
  • run-install: false
  • sfw
  • preview build with vp env doctor and the version assertion
  • component self-test, generated component check, and YAML lint

GitLab verification remains green after the shell-isolation change.

@fengmk2
fengmk2 merged commit a8f9aba into main Aug 20, 2026
51 checks passed
@fengmk2
fengmk2 deleted the agent/xdg-directory-129 branch August 20, 2026 12:49
@fengmk2 fengmk2 mentioned this pull request Aug 24, 2026
fengmk2 added a commit that referenced this pull request Aug 24, 2026
Bump `package.json`, README examples, and the GitLab and Azure template
defaults to `v1.18.0`.

Changes since `v1.17.0`:

- Resolve Vite+ executable paths from `VpDirs` (#131).
- Add GitLab end-to-end tests (#132).
- Clarify automatic Node.js version selection (#130).
- Pin the install script to the requested version (#127).
- Update `pnpm/action-setup` to `v6.0.10` (#124).
- Update `vite-plus` to `v0.2.9` (#125).

After merge, create the annotated tag `v1.18.0` on the merge commit.
Push the tag.
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.

Resolve Vite+ executable paths from VpDirs instead of assuming a fixed bin directory

2 participants