Skip to content

Dev local - #10

Closed
Rowee13 wants to merge 6 commits into
mainfrom
dev-local
Closed

Dev local#10
Rowee13 wants to merge 6 commits into
mainfrom
dev-local

Conversation

@Rowee13

@Rowee13 Rowee13 commented Mar 6, 2026

Copy link
Copy Markdown
Owner

Description

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Test coverage improvement

Related Issues

Closes #

Changes Made

Screenshots (if applicable)

Testing

  • All existing tests pass
  • Added new tests for the changes
  • Tested locally

Checklist

  • My code follows the project's code style
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Additional Notes

Rowee13 and others added 6 commits February 16, 2026 20:20
Runs on PRs to main and pushes to main. Four parallel jobs:
- Lint (turbo lint)
- Type check (turbo check-types)
- API tests (jest)
- Build (turbo build)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Type req.user with proper cast instead of any indexing
- Type req.cookies with Record<string, string>
- Use config.get<string>() for typed config access
- Add eslint-disable for jwtService.decode() which returns any
- Comment out unused csurf import
- Fix unused password destructuring variable

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Fix possibly undefined array access in pagination
- Use Record<string, string> for headers to allow dynamic keys
- Add non-null assertion for JWT parts after length check

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add Prisma client generation to lint job (ESLint TypeScript
  rules need Prisma types to resolve database operations)
- Build @repo/ui components before type-check (docs app imports
  from @repo/ui/* which requires compiled dist/ output)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Allows regular email clients (Gmail, Outlook) to deliver emails
to DevInbox via standard SMTP port 25.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Update @nestjs/core, @nestjs/platform-express, @nestjs/cli, @swc/cli
- Update next in docs app
- Add pnpm overrides for transitive deps (minimatch, lodash, js-yaml,
  ajv, qs, cookie, diff)
- Fix ESLint lint script for ESLint 9 compatibility
- Add .github/dependabot.yml for automated dependency updates
- Result: 0 known vulnerabilities (down from 33)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Rowee13 Rowee13 closed this Mar 6, 2026
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.

1 participant