chore(examples): cleanup and update examples - #389
Merged
Conversation
…from example projects
Bump @forward-software/react-auth to 2.1.2 and typescript to ^7.0.2. TypeScript 7 removes the baseUrl option, so it was dropped from tsconfig.json (paths are already relative, resolution is unaffected).
Bump @forward-software/react-auth to 2.1.2, axios to ^1.20.0, react/ react-dom to ^19.3.0, and typescript to ^7.0.2. TypeScript 7 removes the baseUrl option, so it was dropped from tsconfig.json (paths are already relative, resolution is unaffected).
Bump @forward-software/react-auth to 2.1.2, axios to ^1.20.0, react/ react-dom to ^19.3.0, @vitejs/plugin-react to ^6.1.1, vite to ^8.2.2, and typescript to ^7.0.2. TypeScript 7 removes the baseUrl option, so it was dropped from tsconfig.json (paths are already relative, resolution is unaffected).
Bump @forward-software/react-auth to 2.1.2, react/react-dom to ^19.3.0, @testing-library/jest-dom to ^7.0.1, @testing-library/react to ^16.3.3, @types/react and @types/react-dom to ^19.3.0, @vitejs/plugin-react to ^6.1.1, jsdom to ^30.0.1, typescript to ^7.0.2, vite to ^8.2.2, and vitest to ^5.0.0. Build and test suite verified passing with no code changes required.
Bump @forward-software/react-auth to 2.1.2, react/react-dom to 19.1.0 (required by react-native 0.81.5 peer dep), react-native to 0.81.5, @types/react to ~19.1.10, axios to ^1.20.0, babel-plugin-module-resolver to ^5.0.3, and eslint to ^9.39.5 (kept on 9.x; eslint-config-expo's bundled plugins do not yet support eslint 10). typescript downgraded from ^6.0.0 to ~5.9.2 per 'expo install --check' recommendation for SDK 54 compatibility. eslint-config-expo and @expo/vector-icons left unchanged (already current for this Expo SDK generation / not flagged by expo tooling). Verified with 'expo install --check' (all packages aligned), 'expo-doctor' (18/18 checks passed), and 'tsc --noEmit' (no errors). 'expo lint' fails with a pre-existing EslintPluginImportResolveError unrelated to this change (reproduces identically on the original, unbumped dependency set).
panz3r
added
enhancement
There was a problem hiding this comment.
🟡 Changes recommended
Several example tsconfig.json files now use compilerOptions.paths without baseUrl, which will cause TypeScript configuration errors and break the intended local aliasing.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR refreshes the example apps’ dependencies/tooling (React 19.x, Axios, TypeScript, Vite/Vitest, etc.), aligns all examples on @forward-software/react-auth@2.1.2, and performs some config cleanup to keep the examples current and easier to maintain outside the workspace.
Changes:
- Bumped runtime/dev dependencies across the example projects (React/React DOM, Axios, TypeScript, Vite/Vitest, testing libs).
- Removed
baseUrlfrom multiple exampletsconfig.jsonfiles while keepingcompilerOptions.pathsmappings. - Removed the Showcase example’s
pnpm-lock.yamland added repo-level.gitignorerules for examples’ lockfiles/build outputs.
File summaries
| File | Description |
|---|---|
| examples/showcase/pnpm-lock.yaml | Removed the Showcase example’s pnpm lockfile. |
| examples/showcase/package.json | Updated Showcase example deps/devDeps (React/Auth lib/tooling). |
| examples/reqres/tsconfig.json | Removed baseUrl from TS config while keeping paths mapping. |
| examples/reqres/package.json | Updated ReqRes example deps (auth lib, axios, React) and TS version. |
| examples/refresh-token/tsconfig.json | Removed baseUrl from TS config while keeping paths mapping. |
| examples/refresh-token/package.json | Updated Refresh Token example deps/devDeps (auth lib, axios, tooling). |
| examples/expo/package.json | Updated Expo example deps/devDeps (auth lib, axios, React/RN, lint/TS). |
| examples/base/tsconfig.json | Removed baseUrl from TS config while keeping paths mapping. |
| examples/base/package.json | Updated Base example deps (auth lib) and TS version. |
| .gitignore | Added ignore rules for examples’ lockfiles, node_modules, and dist outputs. |
Review details
Files not reviewed (1)
- examples/showcase/pnpm-lock.yaml: Generated file
- Files reviewed: 8/10 changed files
- Comments generated: 3
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
panz3r
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Affected Package(s)
@forward-software/react-auth(lib)@forward-software/react-auth-google(packages/google-signin)Motivation
This pull request updates dependencies and development tools across several example projects to ensure compatibility with newer versions and to incorporate the latest features and bug fixes. The most significant changes are grouped below by theme.
These updates help keep the example projects current, reduce potential security vulnerabilities, and ensure smoother development and testing workflows.
Description of Changes
Dependency Updates:
@forward-software/react-authto version2.1.2in all example projects for consistency and access to recent improvements. [1] [2] [3] [4] [5]reactandreact-domto the latest19.xversions in all examples, and bumpedreact-nativeto0.81.5in the Expo example. [1] [2] [3] [4]axiosto^1.20.0where applicable. [1] [2] [3]Development Tooling Updates:
typescriptto^7.0.2in most projects (except Expo, which is now at~5.9.2), and updated related type packages. [1] [2] [3] [4] [5]@vitejs/plugin-react,vite,eslint, and testing libraries to their latest versions for improved development experience and compatibility. [1] [2] [3]Configuration Changes:
baseUrlproperty fromtsconfig.jsonin the base, refresh-token, and reqres examples to clean up TypeScript configuration.Breaking Changes
None
How to Test
Vitest) and build steps pass successfully on this PR.pnpm installto install dependencies.pnpm --filter <affected-package> testto run tests for the affected package.pnpm --filter <affected-package> buildto verify the build succeeds.pnpm --filter <affected-package> lintto check for linting errors.Checklist
Notes for Reviewers