Skip to content

Conversation

stevensJourney
Copy link
Collaborator

Overview

This Angular demo isolated test build is currently failing due to a Node.js typing issue.

Error: src/environment.ts:2:16 - error TS2591: Cannot find name 'process'. Do you need to install type definitions for node? Try `npm i --save-dev @types/node` and then add 'node' to the types field in your tsconfig.
2   supabaseUrl: process.env.WEBPACK_PUBLIC_SUPABASE_URL,
                 ~~~~~~~
Error: src/environment.ts:3:16 - error TS2591: Cannot find name 'process'. Do you need to install type definitions for node? Try `npm i --save-dev @types/node` and then add 'node' to the types field in your tsconfig.
3   supabaseKey: process.env.WEBPACK_PUBLIC_SUPABASE_ANON_KEY,
                 ~~~~~~~
Error: src/environment.ts:4:17 - error TS2591: Cannot find name 'process'. Do you need to install type definitions for node? Try `npm i --save-dev @types/node` and then add 'node' to the types field in your tsconfig.
4   powersyncUrl: process.env.WEBPACK_PUBLIC_POWERSYNC_URL

Failed run: https://github.com/powersync-ja/powersync-js/actions/runs/18650828806/job/53168919561

We currently inject environment variables on process.env via a Webpack plugin. The TypeScript compiler errors due to process.env being a Node.js concept, while this is an Angular app which is meant for web.

We could add Node.js types, but that seems strange since this is a web app. Instead this adds a strict definition file for the injected env.

Copy link

changeset-bot bot commented Oct 20, 2025

⚠️ No Changeset found

Latest commit: 82dad63

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@stevensJourney stevensJourney marked this pull request as ready for review October 20, 2025 12:28
@stevensJourney stevensJourney merged commit e549509 into main Oct 20, 2025
9 of 10 checks passed
@stevensJourney stevensJourney deleted the angular-demo branch October 20, 2025 12:28
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