Skip to content

Remove Gitpod integration docs#622

Open
gtsiolis wants to merge 1 commit intomainfrom
doc-226-remove-gitpod-integration-docs
Open

Remove Gitpod integration docs#622
gtsiolis wants to merge 1 commit intomainfrom
doc-226-remove-gitpod-integration-docs

Conversation

@gtsiolis
Copy link
Copy Markdown
Member

See DOC-226.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 30, 2026

Deploying localstack-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9afd984
Status: ✅  Deploy successful!
Preview URL: https://fdfba9ce.localstack-docs.pages.dev
Branch Preview URL: https://doc-226-remove-gitpod-integr.localstack-docs.pages.dev

View logs

@quetzalliwrites
Copy link
Copy Markdown
Collaborator

quetzalliwrites commented May 4, 2026

Hey @gtsiolis ty again for your contributions :) , did you realize your build is failing in this PR?

The build is crashing because astro.config.mjs (Line 41) is trying to call .replace() on an undefined variable that is undefined.

It's possible this is also related to the package-lock.json issue I explained to you last week.

  1. I suspect this is possibly related because the error reads as if there's missing config keys in the build dashboard.

  2. Also suspect: If the package-lock.json file issue caused a version mismatch in Vite/Astro, it might be passing unexpected arguments to the config file.

BUILD ERROR

17:57:11.532 | [astro] Unable to load your Astro config
-- | --
17:57:11.533 |  
17:57:11.755 | Cannot read properties of undefined (reading 'replace')
17:57:11.755 | Location:
17:57:11.755 | /opt/buildhome/repo/node_modules/astro/node_modules/vite/dist/node/module-runner.js:913:3
17:57:11.755 | Stack trace:
17:57:11.756 | at eval (/opt/buildhome/repo/astro.config.mjs:41:37)
17:57:11.756 | at async ESModulesEvaluator.runInlinedModule (file:///opt/buildhome/repo/node_modules/astro/node_modules/vite/dist/node/module-runner.js:913:3)
17:57:11.756 | at async ModuleRunner.cachedRequest (file:///opt/buildhome/repo/node_modules/astro/node_modules/vite/dist/node/module-runner.js:1053:73)
17:57:11.756 | at async loadConfigWithVite (file:///opt/buildhome/repo/node_modules/astro/dist/core/config/vite-load.js:42:19)
17:57:11.756 | at async resolveConfig (file:///opt/buildhome/repo/node_modules/astro/dist/core/config/config.js:86:22)
17:57:11.846 | Failed: Error while executing user command. Exited with error code: 1
17:57:11.854 | Failed: build command exited with code: 1
17:57:12.586 | Failed: error occurred while running build command

</pre>

Copy link
Copy Markdown
Collaborator

@quetzalliwrites quetzalliwrites left a comment

Choose a reason for hiding this comment

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

gotta fix yo build, amigo @gtsiolis 😸

@gtsiolis gtsiolis force-pushed the doc-226-remove-gitpod-integration-docs branch from d14dd60 to 2527c7c Compare May 4, 2026 11:58
@gtsiolis
Copy link
Copy Markdown
Member Author

gtsiolis commented May 4, 2026

Thanks, @quetzalliwrites!

This seems to be a configuration issue with Astro, not related to package-lock.json. Added 2527c7c to resolve this.

I'd have never guessed build is failing locally as running build locally. Could we add build in GH Actions for pre-check? Added #632 to add a check. Cc @remotesynth

@remotesynth
Copy link
Copy Markdown
Collaborator

remotesynth commented May 4, 2026

@quetzalliwrites @gtsiolis the build worked. I just had rerun. This was a Cloudflare issue not an issue with the PR. I am worried about making changes to the Astro config on a PR that is just a content update

@remotesynth
Copy link
Copy Markdown
Collaborator

Hey @gtsiolis ty again for your contributions :) , did you realize your build is failing in this PR?

The build is crashing because astro.config.mjs (Line 41) is trying to call .replace() on an undefined variable that is undefined.

It's possible this is also related to the package-lock.json issue I explained to you last week.

1. I suspect this is possibly related because the error reads as if there's **missing config keys in the build dashboard**.

2. Also suspect: If the `package-lock.json` file issue caused a version mismatch in Vite/Astro, it might be passing unexpected arguments to the config file.

BUILD ERROR

17:57:11.532 | [astro] Unable to load your Astro config
-- | --
17:57:11.533 |  
17:57:11.755 | Cannot read properties of undefined (reading 'replace')
17:57:11.755 | Location:
17:57:11.755 | /opt/buildhome/repo/node_modules/astro/node_modules/vite/dist/node/module-runner.js:913:3
17:57:11.755 | Stack trace:
17:57:11.756 | at eval (/opt/buildhome/repo/astro.config.mjs:41:37)
17:57:11.756 | at async ESModulesEvaluator.runInlinedModule (file:///opt/buildhome/repo/node_modules/astro/node_modules/vite/dist/node/module-runner.js:913:3)
17:57:11.756 | at async ModuleRunner.cachedRequest (file:///opt/buildhome/repo/node_modules/astro/node_modules/vite/dist/node/module-runner.js:1053:73)
17:57:11.756 | at async loadConfigWithVite (file:///opt/buildhome/repo/node_modules/astro/dist/core/config/vite-load.js:42:19)
17:57:11.756 | at async resolveConfig (file:///opt/buildhome/repo/node_modules/astro/dist/core/config/config.js:86:22)
17:57:11.846 | Failed: Error while executing user command. Exited with error code: 1
17:57:11.854 | Failed: build command exited with code: 1
17:57:12.586 | Failed: error occurred while running build command

</pre>

I had verified that it was not a package-lock.json issue as that file was not even updated in the PR. This was purely an issue with Cloudflare. I am not sure why this issue keeps happening and why it seems limited to our preview deploys, but it's unrelated to this PR.

@quetzalliwrites
Copy link
Copy Markdown
Collaborator

fun fun fun day, today @gtsiolis 😸

so, as Brian already explained/fixed the build issue.. your PR should be ok now

that said, we shouldn't be making changes to the astro.config.mjs... can you please revert and undo that commit?

@gtsiolis gtsiolis force-pushed the doc-226-remove-gitpod-integration-docs branch from 2527c7c to 9afd984 Compare May 4, 2026 13:29
@gtsiolis
Copy link
Copy Markdown
Member Author

gtsiolis commented May 4, 2026

DONE

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.

3 participants