Skip to content

fix: update publish workflow - #22

Merged
ryanbaumann merged 4 commits into
mainfrom
fix-publish
Aug 11, 2025
Merged

ryanbaumann merged 4 commits into
mainfrom
fix-publish

Conversation

@ryanbaumann

Copy link
Copy Markdown
Collaborator

Fix github release workflow

  1. Ensure dependencies are installed from the public registry before configuring the environment for publishing via WDR.
  2. Setup Bun prior to install

@ryanbaumann

ryanbaumann commented Aug 10, 2025 •

Copy link
Copy Markdown
Collaborator Author

Edit - this PR fixes #21 . However, the publish workflow still won't complete b/c but we admin change to make to ensure the pipeline runs. In the meantime I made a manual push to NPM with version bump to 0.1.4 (actually now 0.1.5 since there was another build error I discovered and fixed in the lastest commit in this branch) as specified in this PR.

Let's address Monday @caio1985

…ncies

The `npx @googlemaps/code-assist-mcp` command was failing for users due to two issues in the packaging process.

First, the bundled output file in `dist/index.js` was not being marked as executable, leading to a `Permission denied` error when `npx` tried to run it.

Second, the `express` and `cors` packages were listed as `devDependencies` but are required at runtime. This would have caused a "Cannot find module" error after fixing the permissions.

This commit addresses both issues by:
- Adding `shx chmod +x dist/index.js` to the `build:prepare` npm script.
- Moving `express` and `cors` to the `dependencies` section in the package's `package.json`.
@ryanbaumann
ryanbaumann requested a review from caio1985 August 11, 2025 20:40

@caio1985 caio1985 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@ryanbaumann
ryanbaumann merged commit 1088cc7 into main Aug 11, 2025
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