Skip to content

Feat/connections page redesign#11

Merged
davidancor merged 3 commits intomainfrom
feat/connections-page-redesign
Feb 21, 2026
Merged

Feat/connections page redesign#11
davidancor merged 3 commits intomainfrom
feat/connections-page-redesign

Conversation

@davidancor
Copy link
Contributor

Description

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • 🎨 UI/UX improvement
  • ⚡ Performance improvement
  • ♻️ Code refactoring
  • 🧪 Test addition or update
  • 🔧 Configuration change

Related Issues

Closes #

Changes Made

Testing

Test Coverage

  • Unit tests added/updated
  • Integration tests added/updated
  • E2E tests added/updated
  • Manual testing completed

Test Steps

Test Results

# Example: paste test output here

Documentation

  • Updated relevant README files
  • Updated API documentation
  • Updated agent instructions (.claude/teams/)
  • Created/updated ADR (if architecture change)
  • Updated CHANGELOG.md
  • Added code comments for complex logic
  • Updated environment variable documentation

Screenshots/Videos

Checklist

Code Quality

  • Code follows project style guidelines
  • TypeScript types are properly defined
  • No console.log or debug code left in
  • Error handling is implemented
  • Code is DRY (Don't Repeat Yourself)

Testing

  • All tests pass locally
  • Test coverage maintained or improved
  • Edge cases are covered

Documentation

  • CHANGELOG.md updated (unless skip-changelog label)
  • Relevant documentation updated
  • Comments added for complex logic

Security

  • No sensitive data exposed
  • Input validation implemented
  • Authentication/authorization checked
  • Dependencies are secure (no known vulnerabilities)

Performance

  • No unnecessary re-renders (React)
  • Database queries optimized
  • No N+1 query issues
  • Bundle size impact considered

Accessibility

  • ARIA labels added where needed
  • Keyboard navigation works
  • Color contrast is sufficient
  • Screen reader tested (if UI change)

Breaking Changes

Impact:

Migration Path:

Deployment Notes

  • Database migration required
  • Environment variables added/changed
  • External service configuration needed
  • Cache needs to be cleared
  • Other (describe):

Rollback Plan

Agent Team Review

Requesting review from:

Additional Context


Reviewer Guidelines

What to Look For

  1. Correctness: Does the code do what it claims?
  2. Testing: Are there adequate tests?
  3. Documentation: Is it well-documented?
  4. Performance: Any performance concerns?
  5. Security: Any security issues?
  6. Maintainability: Is the code maintainable?

Approval Criteria

  • ✅ All CI checks pass
  • ✅ Code review approval from at least 1 team member
  • ✅ Documentation updated
  • ✅ Tests pass and coverage maintained
  • ✅ No unresolved comments

Adds atomic credit check-and-deduct system to gate AI generation.
Free tier gets 2,500 credits (~2 projects); paid subscribers get
unlimited. Credits reset on subscription status change via Stripe
webhooks. Frontend handles 402 with upgrade prompts (dialog for
Quick Start, toast for chat).

Credit costs: Quick Start 1250, chat message 5, agent regen 100.
…line

Passes onboarding metadata (project type, stage, budget) to all 6
generator agents for KB-aware output. Adds infrastructure and coding
guidelines generation to Quick Start pipeline. Adds Haiku-based
project context inference for auto-detecting project type.
@vercel
Copy link

vercel bot commented Feb 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
c1v-product-helper-9hdq Error Error Feb 21, 2026 5:07pm

Request Review

@davidancor davidancor merged commit 9bb9f5d into main Feb 21, 2026
5 of 13 checks passed
davidancor added a commit that referenced this pull request Feb 23, 2026
PR #11 merge duplicated checkAndDeductCredits imports and credit gate
blocks in 7 API routes. Also add missing planName to query columns
in checkAndDeductCredits.
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