Conversation
- Switch packageManager from bun to npm - Update scripts to use turbo directly (no pnpm prefixes) - Restore workspaces field for npm - Clean up all workspace:* references to * - Delete pnpm-workspace.yaml - Delete .npmrc - Generate package-lock.json - Update tooling package references
- Switch from npm to pnpm package manager - Add pnpm-workspace.yaml for workspace management - Update all @query/* internal package references to workspace:* - Add pnpm-lock.yaml - Update CI workflows to use pnpm - Add pnpm-ci.yml for CI pipeline - Add CODEOWNERS to ensure aamoghS reviews PRs
- Add pnpm/action-setup step to CI workflows - Use pnpm cache for node_modules - Remove bun setup since we use pnpm now - Add node cache setup to speed up builds
- Create proxy.ts middleware for edge/CDN caching
- Add Cache-Control headers for static and dynamic content
- Add ETag and Last-Modified headers for validation caching
- Add Vary headers for cache correctness
- Add security headers (X-Content-Type-Options, X-Frame-Options, X-XSS-Protection)
- Configure cache duration per route pattern (static assets, pages, dynamic content)
- Also add author attribution to layout files:
- mainweb/app/layout.tsx: authors: [{ name: aamoghS }]
- hacklytics2027/app/layout.tsx: authors: [{ name: aamoghS }]
- Optimize React Query configuration:
- Increase staleTime from 60s to 5 minutes
- Add gcTime for better cache reuse
- Enable background updates
- Set offlineFirst network mode
- Add placeholder='blur' to images for better perceived performance
- Truncate SVG path decimals in turborepo-logo to 2 decimal places - Improve SVG rendering performance - Fix all react-doctor issues (score: 100/100) - Complete HTTP caching middleware implementation - Add author attribution to all layouts
- Document how to restrict commits to 'aamoghS' via GitHub branch protection - Requires admin access to repo settings: Settings > Branches > main
- Remove console.log from sendVerificationRequest - Remove unused 'url' parameter from redirect callback - Keep console.error for actual errors (not removed)
NextAuth's redirect callback uses 'url' (not '_url') as the parameter name. TypeScript compiles correctly with 'url' since the unused-args rule allows '_' prefixed parameters.
- Remove console.warn from adapter initialization - Remove console.error from adapter catch block - Remove unused 'error' variable in catch block - Keep error throwing for proper error handling
- Add eslint, @typescript-eslint to @query/auth devDependencies - Update pnpm action to v5 (latest) - Pin pnpm version to 10 in CI This ensures lint passes on CI without needing global eslint installation.
Let pnpm@10.33.2 from package.json determine the version. This avoids the 'multiple versions specified' error from the pnpm action self-installer.
Turbo is required for the build command. Using catalog dependency pattern for workspace packages.
The site's next.config.ts enables React Compiler. babel-plugin-react-compiler is now installed to satisfy this requirement during build.
Feature: feature/member-site to dev
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
1 participant
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.
Automated PR tracking changes from
devintomain.