Conversation
…esk in layout.tsx
Button: Tangerine bg, Figma shadow via var(), 12px radius, 44px height, secondary=black/5, outline=transparent, ghost=hover bg, link=no bg. Input: bg-black/5, border rgba(13,13,13,0.15), 12px radius, 40px height, placeholder opacity via rgba, dark mode white/15. Card: white bg, subtle border rgba(13,13,13,0.15), shadow 0px 1px 2px. Badge: 6px radius (rounded-md), bg-black/5, proper dark mode variants.
… accordion, dropdown-menu, avatar to Figma specs Tabs: container 10px radius with border, active tab 8px radius white bg. Select: trigger matches input (black/5, 12px radius), content white bg. Dialog: white bg, rgba border, 12px radius, removed inline style overrides. Sheet: white bg, rgba border colors for light/dark. Label: text-base leading-[150%] (Consolas via body inheritance). Radio-group: 18x18 size, unchecked=black/5 border, checked=solid black/white. Accordion: border rgba(13,13,13,0.15), text-base trigger. Dropdown-menu: white bg, rgba border, 12px radius, items rounded-lg text-base. Avatar: added rgba border for light/dark modes. Collapsible: no changes needed (inherits tokens correctly).
…ma tokens
Replace inline style={{}} with Tailwind token classes across 5 components:
- explorer-sidebar: bg-sidebar, border-sidebar-border, text-sidebar-foreground
- project-context-card: text-foreground, text-muted-foreground, border-border
- quick-instructions: bg-primary, text-primary-foreground, border-border
- artifact-pipeline: text-foreground, text-muted-foreground, text-green-500
- project-header-compact: bg-background, border-border, text-foreground
All fontFamily inlines removed. Only dynamic layout styles remain as inline.
…tion components
Remove all inline style={{ }} from the 6 heaviest section components
(guidelines, schema, system-overview, api-spec, architecture, user-stories).
Font-family inlines removed (Consolas inherited from body, Space Grotesk
from headings). Color/background/border values replaced with Tailwind
token classes: text-foreground, text-muted-foreground, text-primary,
bg-muted, bg-card, border-border. Total inline styles in these 6 files
reduced from 193 to 0.
…ma tokens
Replace inline style={{}} with Tailwind token classes across 5 components:
- explorer-sidebar: bg-sidebar, border-sidebar-border, text-sidebar-foreground
- project-context-card: text-foreground, text-muted-foreground, border-border
- quick-instructions: bg-primary, text-primary-foreground, border-border
- artifact-pipeline: text-foreground, text-muted-foreground, text-green-500
- project-header-compact: bg-background, border-border, text-foreground
All fontFamily inlines removed. Only dynamic layout styles remain as inline.
…okens
Replace inline style={{ }} with Tailwind utility classes in welcome-onboarding,
pipeline-card, progress-cards, and quick-start-button. Pipeline card uses
bg-card border-border rounded-xl with Tangerine primary buttons. Progress card
step icons use token colors (text-primary, text-destructive). Quick start button
removes all style overrides — inherits from updated shadcn Button component.
Onboarding header uses text-foreground/text-muted-foreground tokens.
- Add PLAN_LIMITS constants with 10% grace period (Free=2750, Base=5500, Plus=unlimited) - Add creditsUsed, creditLimit, teamMemberLimit columns to teams schema - Replace handleSubscriptionChange with per-tier limit assignment - Add checkAndDeductCredits with Base tier enforcement (was bypassing all paid) - Add team member limit check to inviteTeamMember - Set tier limits on checkout success route - Add customer.subscription.created webhook event - Remove trial_period_days from checkout session - Update pricing page: Free Trial -> Free, credit-based copy, conditional trial text - Generate migration 0007 with backfill for existing teams
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
Here's the filled-out PR template for the tier enforcement work:
Type:
Related Issues
Closes # (no issue tracked — this came from the pricing spec)
Changes Made
Base=5,500, Plus=unlimited)
migration 0007
Stripe product name
atomic race-safe SQL
Plus=unlimited)
trial text
Testing
Test Coverage
Test Steps
teamMemberLimit
Test Results
PASS lib/constants/tests/plan-limits.test.ts (9 tests)
PASS lib/payments/tests/stripe-tiers.test.ts (3 tests)
PASS lib/db/tests/credit-enforcement.test.ts (6 tests)
PASS app/(login)/tests/invite-limits.test.ts (6 tests)
Test Suites: 2 failed, 21 passed, 23 total
Tests: 2 failed, 640 passed, 642 total
(2 failures are pre-existing — missing ANTHROPIC_API_KEY in core-tools.test.ts)
Documentation
Checklist
Code Quality
Testing
Documentation
Security
Performance
Accessibility
Breaking Changes
Impact: None. New columns have defaults matching existing behavior (free tier
limits). Existing teams unaffected until migration backfill runs.
Migration Path: Apply migration 0007 via Supabase SQL Editor before merge.
Deployment Notes
Supabase SQL Editor
(case-sensitive)
Rollback Plan
COLUMN IF EXISTS credit_limit, DROP COLUMN IF EXISTS team_member_limit;
Agent Team Review
Additional Context
Tier limits: Free (2,500 credits / 2 members), Base CA$19.99 (5,000 credits / 2
members), Plus CA$49.99 (unlimited). Grace period of 10% on credit limits for
soft cap behavior. Stripe prices still placeholder ($8/$12 USD) — not touched
in this PR per David's instruction.