Skip to content

Conversation

@mavisakalyan
Copy link

Pull Request

Description

Related Issue

Closes #issue_number

Linear Ticket: ALT-XXX

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
  • Security fix
  • Dependency update
  • CI/CD change

Changes Made

Screenshots / Recordings

Testing

Test Coverage

  • Unit tests added/updated
  • Integration tests added/updated
  • All existing tests pass
  • Test coverage maintained or improved

Manual Testing

Describe how you tested these changes:

Test Environment

  • OS:
  • Node.js:
  • PostgreSQL:
  • Redis:

Checklist

Code Quality

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have removed any console.log statements or debug code
  • My changes generate no new warnings or errors
  • My code passes TypeScript type checking

Documentation

  • I have updated the documentation (if applicable)
  • I have updated CHANGELOG.md
  • I have added/updated JSDoc comments for new functions
  • I have updated GraphQL schema descriptions (if applicable)
  • I have updated README.md (if applicable)

Database Changes

  • I have created necessary Prisma migrations
  • Migrations are backwards compatible (or breaking change is documented)
  • I have tested migrations on a copy of production data (if applicable)
  • I have updated seed data (if applicable)

Security & Privacy

  • I have considered security implications of my changes
  • I have not introduced any security vulnerabilities
  • I have not exposed any sensitive data or credentials
  • My changes maintain or improve user privacy
  • I have considered censorship resistance implications
  • I have run npm audit and addressed any critical issues

Dependencies

  • I have minimized new dependencies
  • New dependencies are from trusted sources
  • I have documented why new dependencies are needed
  • I have checked licenses of new dependencies (compatible with GPLv3)
  • Dependencies are up-to-date and have no known vulnerabilities

Performance

  • I have considered performance implications
  • My changes do not significantly impact load times or resource usage
  • I have optimized database queries (if applicable)
  • I have considered caching strategies (if applicable)

Deployment

  • My changes work in a Docker container
  • I have tested on Akash deployment (if applicable)
  • I have updated environment variable documentation (if applicable)
  • I have considered backward compatibility

Breaking Changes

Deployment Notes

Additional Context

Privacy & Security Impact

Reviewer Notes


By submitting this pull request, I confirm that:

@claude
Copy link

claude bot commented Jan 20, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

@claude
Copy link

claude bot commented Jan 22, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

@claude
Copy link

claude bot commented Jan 22, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

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