summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
39 hoursImprove dev mode login UXBen Sima
- Add mx-1 (horizontal margin) to code element for better text spacing - Pre-fill email input with demo@example.com in dev/test mode - Makes it faster to test login flow in development Amp-Thread-ID: https://ampcode.com/threads/T-8edacbeb-b343-49ca-b524-1c999272acb6 Co-authored-by: Amp <amp@ampcode.com>
39 hoursAdd dev taskBen Sima
39 hoursImprove type safety in Billing moduleBen Sima
- Add explicit type annotations to TIER_LIMITS dict - Add type annotation to tier_info dict in get_tier_info - Remove unnecessary type: ignore comments - Use .get() consistently for dict access Improves type safety and removes 3 type: ignore suppressions. Amp-Thread-ID: https://ampcode.com/threads/T-8edacbeb-b343-49ca-b524-1c999272acb6 Co-authored-by: Amp <amp@ampcode.com>
39 hoursExtract format_duration utility to UI moduleBen Sima
Moved format_duration function from Web.py to UI.py for better code organization. This is a UI utility function used for displaying episode durations, so it belongs in the shared UI module rather than the web-specific module. Amp-Thread-ID: https://ampcode.com/threads/T-8edacbeb-b343-49ca-b524-1c999272acb6 Co-authored-by: Amp <amp@ampcode.com>
39 hoursReplace bare exception catches with specific exceptionsBen Sima
- Replace Exception with httpx.HTTPError, httpx.TimeoutException, re.error in extract_og_metadata - Replace Exception with ValueError, KeyError in auth verification - Replace Exception with httpx errors and ValueError in submit_article - Replace Exception with ValueError, KeyError, AttributeError in RSS feed generation - Replace Exception with ValueError, KeyError in cancel/retry/delete job handlers Improves error handling specificity and removes BLE001 linter suppressions. Amp-Thread-ID: https://ampcode.com/threads/T-8edacbeb-b343-49ca-b524-1c999272acb6 Co-authored-by: Amp <amp@ampcode.com>
39 hoursUpdate agents and plan out next workBen Sima
41 hoursSimplify billing to single paid planBen Sima
- Remove /billing page and BillingPage component - Add callout box on home page showing articles remaining for free users - Upgrade Now button goes directly to Stripe checkout - Change tier from 'pro' to 'paid' throughout - Update redirect URLs to go to / instead of /billing - Remove Billing button from navbar, add Manage Account link for all users - Add /account route with coming soon page - Hide payment banner for paid users Amp-Thread-ID: https://ampcode.com/threads/T-7de89e42-947c-4243-be19-0cb75be607e7 Co-authored-by: Amp <amp@ampcode.com>
42 hoursSwitch to just one paid planBen Sima
I don't need the complexity of multiple plans, just one for now to test.
42 hoursFix tests I thinkBen Sima
amp worked on this overnight, and I cleaned it up a bit this morning. I think its all correct now.
42 hoursMove end to end test to own fileBen Sima
This is a big test, and probably over-dependencied, but it's nice to have an end-to-end test as an overall bug-catcher.
42 hoursCleanup some logging setup codeBen Sima
I think the calls to Log.setup() were accidentally creating multiple loggers, hopefully this fixes the problem.
2 daysPodcastItLater end to end testBen Sima
2 daysDocument database migrations and external API testing patternsBen Sima
Add to AGENTS.md: - SQLite migration limitations (no UNIQUE constraints via ALTER TABLE) - Migration logging best practices - Testing external API integrations (Stripe pattern) - Mock webhook testing approach - Local webhook testing with CLI tools - Handling API version differences with fallback field names - Database inspection for debugging integrations These patterns emerged from implementing and debugging the Stripe billing integration.
2 daysUpdate webhook test to match real Stripe API behaviorBen Sima
- Use billing_cycle_anchor instead of current_period_start/end in test - Reflects actual Stripe API v2025-10-29 subscription structure - Test validates upgrade flow works with newer API versions - Removes debug logging (list of available keys)
2 daysFix subscription webhook to handle Stripe API version differencesBen Sima
- Use billing_cycle_anchor or start_date as fallback for period start - Calculate period end by adding 1 month if not provided - Handles newer Stripe API versions that don't include current_period_* fields - Fixes issue where subscription.created webhook couldn't update user plan - Works with Stripe API Version 2025-10-29.clover and newer
2 daysAdd detailed logging for missing subscription period datesBen Sima
Log available keys when period dates are missing to help debug why subscription webhooks fail to update user plan.
2 daysAdd full checkout flow test with proper client_reference_idBen Sima
- Test simulates complete Stripe checkout flow end-to-end - Step 1: checkout.session.completed with client_reference_id - Step 2: customer.subscription.created with subscription details - Verifies customer linking and plan upgrade to pro tier - Tests ensure webhook flow works without calling real Stripe API - Validates that client_reference_id properly links session to user
2 daysFix TypeError in checkout webhook when client_reference_id is NoneBen Sima
- Handle case where client_reference_id is None or missing - Fall back to checking metadata.user_id if client_reference_id not available - Add try/except for int() conversion with proper error logging - Use safe .get() for session ID in error messages - Prevents TypeError: int() argument must be a string when webhook has None value
2 daysAllow webhook signature verification bypass for local testingBen Sima
- Skip signature verification if STRIPE_WEBHOOK_SECRET is not set - Add warning log when verification is skipped - Parse webhook payload as JSON directly in test mode - Enables local testing with 'stripe trigger' without configuring webhook secret - Production still requires proper webhook secret for security
2 daysFix billing migration to work with SQLite ALTER TABLE limitationsBen Sima
- Remove UNIQUE constraints from ALTER TABLE statements (SQLite doesn't support adding constraints via ALTER TABLE) - Add better logging for migration failures (debug level) - Rely on application logic to ensure uniqueness instead of DB constraint - This fixes the silent failure where stripe_customer_id and stripe_subscription_id columns weren't being added
3 daysAdd Stripe webhook tests and testing documentationBen Sima
- Add TestWebhookHandling class with tests for subscription webhooks - Test normal subscription creation flow with mock data - Test handling of incomplete webhook data (missing fields) - Add STRIPE_TESTING.md with comprehensive testing guide: - How to use Stripe test mode and test cards - How to use Stripe CLI to trigger test webhooks - Instructions for running unit tests - Database migration instructions for production - End-to-end testing workflow - Common troubleshooting issues Tests verify webhook handling works without calling real Stripe API.
3 daysFix Stripe webhook KeyError by using safe dict accessBen Sima
- Replace direct dict key access with .get() for all subscription fields - Add validation and early return for missing required fields - Add logging warnings when subscription data is incomplete - Handles both dict and Stripe object access patterns safely - Fixes KeyError: 'current_period_start' when processing webhooks
3 daysMake navbar collapsible on mobile with togglerBen Sima
- Add hamburger toggle button positioned on right side - Navbar collapses on small screens (< lg breakpoint) - Mobile view shows vertically stacked elements: - Logged in as: email - Free: X articles left [Upgrade Now] (free users) - Billing (paid users only) - Admin Queue (admins only) - Logout - Remove 'Plan: Pro' text for paid users (just show Billing button) - Add rounded corners to navbar with 'rounded' class - Remove 1px bottom border in dark mode - Toggle controlled by Bootstrap's collapse component
3 daysAdd dark mode styling for navbarBen Sima
- Navbar now uses dark background (#2b3035) in dark mode - Add border-bottom to separate navbar from content in dark mode - Ensure navbar-text elements have proper light color (#dee2e6) in dark mode - Maintains light mode appearance with bg-body-tertiary
3 daysConvert user banner to horizontal navbar and make Upgrade btn-successBen Sima
- Change Upgrade Now button from btn-warning to btn-success - Convert user info card to Bootstrap navbar with horizontal layout - Free tier: quota text and Upgrade Now button inline with navbar-text styling - Paid tier: plan name displayed inline as navbar-text - Email, plan info, and action buttons all in single horizontal bar - Uses bg-body-tertiary for automatic light/dark mode compatibility
3 daysMake Upgrade Now an actual button in free tier bannerBen Sima
- Replace '[Upgrade Now]' text link with proper btn-warning button - Add arrow-up-circle icon to Upgrade Now button - Hide Billing button for free tier users (they now have Upgrade Now) - Keep Billing button visible for paid tier users
3 daysRefactor UI components and add dark mode to admin pagesBen Sima
- Create shared UI module (Biz/PodcastItLater/UI.py) with: - create_bootstrap_styles() - create_auto_dark_mode_style() - create_htmx_script() - create_bootstrap_js() - Update Admin.py to use shared UI module and add dark mode support - Update Web.py to use shared UI module - Admin Queue and User Management pages now support automatic dark mode
3 daysMake RSS subscribe link inline with copy-to-clipboardBen Sima
Subscribe link now appears inline next to label. Click copies URL to clipboard and shows 'Copied!' feedback for 2 seconds. Link truncates if too long to fit in bounding box.
3 daysAdd usage banner for free tier users on home pageBen Sima
Free tier users now see 'Free: X articles left [Upgrade Now]' banner. Paid users see plan name with icon as before. Banner replaces the old plan display in user info card.
3 daysRemove personal tier, keep only free and proBen Sima
Simplified pricing to two tiers: - Free: 10 articles total (lifetime) - Pro: $29/month unlimited articles Removed STRIPE_PRICE_ID_PERSONAL from configuration.
3 daysChange free plan from monthly to lifetime limitBen Sima
Free tier now has 10 articles total (lifetime) instead of 10 per month. Period boundaries for free users now span from account creation to far future.
3 daysRemove manual approval requirement for new accountsBen Sima
New accounts now default to 'active' status instead of 'pending'. Users can start using the service immediately after signup.
3 daysAdd complete Stripe billing integration to PodcastItLaterBen Sima
- Implement Biz.PodcastItLater.Billing with checkout sessions, billing portal, webhook handling - Add subscription database schema: plan_tier, stripe fields, period dates, stripe_events table - Three-tier pricing: free (10/month), personal (/month, 50 articles), pro (9/month, unlimited) - Usage tracking and enforcement with tier-based limits - Full billing UI with plan display, usage stats, pricing cards, upgrade buttons - Dashboard shows current tier with billing button - Update Web.nix with Stripe environment variables - Fix POST redirects to Stripe with 303 status code for CloudFront compatibility Amp-Thread-ID: https://ampcode.com/threads/T-c139e5b5-1901-4cd6-8030-5623bfe1df35 Co-authored-by: Amp <amp@ampcode.com>
3 daysFix exit condition for run.shBen Sima
Without this, run.sh would continue if bild failed, and you might end up running an out of date artifact.
3 daysDisable storybook for now - not working on itBen Sima
5 daysRemove bg-light from user info card for proper dark modeBen Sima
The bg-light class was forcing the 'Logged in as' card to stay light even in dark mode. Removing it allows Bootstrap's default card background to adapt to dark mode.
5 daysFix dark mode CSS variablesBen Sima
- Set all CSS variables on :root instead of html/body - Added RGB variants needed by Bootstrap - Added secondary, tertiary, and border color variants - Removed unnecessary light mode media query This should now properly apply dark theme when system prefers dark mode.
5 daysPodcastItLater: Fix dark mode - use automatic CSS-only approachBen Sima
- Removed JavaScript theme switcher (not needed) - Removed toggle buttons from all pages - Added automatic dark mode CSS based on prefers-color-scheme media query - Added color-scheme meta tag for native UI hints - Uses inline CSS instead of external file (build system constraint) - Zero JavaScript - pure CSS solution - Automatically follows system/browser dark mode preference Task: t-64tkB5
5 daysPodcastItLater: Add dark mode supportBen Sima
- Added theme switcher script with localStorage persistence - Respects system dark mode preference (prefers-color-scheme) - Added theme toggle button to HomePage and BillingPage - Uses Bootstrap 5.3 data-bs-theme attribute for dark mode - Theme persists across page loads - Icon changes between moon (light) and sun (dark) Task: t-64tkB5
5 daysPodcastItLater: Add Stripe billing and mobile responsivenessBen Sima
- Implemented complete Stripe integration (Billing.py) - Checkout sessions for subscription upgrades - Billing portal for subscription management - Webhook handling for subscription events - Usage tracking with tier-based limits (free: 10, personal: 50, pro: unlimited) - Added billing page UI (BillingPage component) - Current plan display with usage stats - Pricing cards for all tiers with upgrade buttons - Manage subscription button for paid users - Success/error messaging - Database migrations for billing - Added plan_tier, stripe_customer_id, stripe_subscription_id - Added subscription_status, period dates, cancel_at_period_end - Created stripe_events table for webhook idempotency - Added get_usage() method for usage tracking - Made UI mobile-friendly and responsive - Added viewport meta tags to all pages - Replaced pages.HtmlPage with raw html.html for meta tag control - Responsive button layouts with flexbox wrapping - Responsive pricing cards (1 col mobile, 2 col tablet, 3 col desktop) - Touch-friendly forms and buttons (44px minimum) - Responsive padding and containers - Admin tables with horizontal scroll - Added Stripe testing guide (STRIPE_TESTING.md) - Fixed CSS bug in pricing cards (cardh-100 text rendering) - Updated tasks: completed t-144e7lF, t-1pIV0ZF, t-1s8ADC0 Amp-Thread-ID: https://ampcode.com/threads/T-42fd5fb3-3dc5-4cbc-a9a3-78db9e13187e Co-authored-by: Amp <amp@ampcode.com>
6 daysfeat(PodcastItLater): Add usage limit enforcement and billing UIBen Sima
- Enforce tier limits before article submission - Display current plan in user info card (Free/Personal/Pro) - Add Billing button to dashboard navigation - Show friendly upgrade prompt when limit reached - Link to /billing page for plan management Limits enforced: - Free: 10 articles/month - Personal: 50 articles/month - Pro: Unlimited Related to task t-144e7lF
6 daysdocs: Document task management and typecheck conventions in AGENTS.mdBen Sima
- Document using 'task create' instead of TODO/FIXME comments - Document using Omni/Ide/typecheck.sh for fast Python typechecking - Create task t-1pIV0ZF for billing page UI implementation
6 daysfeat: Add stripe to Python deps and document dependency processBen Sima
- Add stripe to Omni/Bild/Deps/Python.nix (alphabetically sorted) - Fix all type annotations in Billing.py for mypy - Document how to add Python packages in AGENTS.md - Add billing routes to Web.py (checkout, portal, webhook) This enables Stripe integration in PodcastItLater. Related to task t-144e7lF
6 daysfeat(PodcastItLater): Add Stripe billing infrastructureBen Sima
Add complete Stripe integration backend ready for testing once stripe package is available in Nix environment. Components: - Billing.py: Stripe Checkout, Billing Portal, webhook handling - Database migrations for subscription tracking - Usage tracking with tier-based limits - Idempotent webhook processing with stripe_events table Tier limits: free (10/mo), personal (50/mo), pro (unlimited) Webhook events handled: - checkout.session.completed (link customer to user) - customer.subscription.{created,updated,deleted} (sync state) - invoice.payment_failed (mark past_due) Requires: Stripe Python package in Nix, Web.py routes (next commit) Related to task t-144e7lF Amp-Thread-ID: https://ampcode.com/threads/T-8feaca83-dcc2-46cb-8f71-d0785960a2f7 Co-authored-by: Amp <amp@ampcode.com>
6 daysrefactor(PodcastItLater): Clean up RSS configBen Sima
RSS feed already generates personalized titles from user email. Remove unused hardcoded values and change author to PodcastItLater.
6 daysfix(PodcastItLater): Remove audio player, move RSS to episodes sectionBen Sima
- Remove audio player from Recent Episodes (use podcast app instead) - Move RSS feed link from user info card to Recent Episodes section - Fix audio playback interruption on refresh - Simplify user info card to just show email and logout button This fixes the UX issue where dashboard refresh would interrupt playing audio. Completes task t-17Z0069
6 daysfeat(PodcastItLater): Apply Bootstrap 5 UI and fix dev loginBen Sima
- Apply Bootstrap 5 CSS and icons to all pages (Web.py, Admin.py) - Convert all components to use Bootstrap classes instead of inline styles - Add dev mode banner showing demo@example.com for instant login - Implement secure demo account (demo@example.com) with auto-approval - Fix HTMX loading issue when load_styles=False - Update Database.create_user() to accept optional status parameter - Add Bootstrap tables, cards, badges, and button groups throughout - All tests passing Completes task t-144drAE Amp-Thread-ID: https://ampcode.com/threads/T-8feaca83-dcc2-46cb-8f71-d0785960a2f7 Co-authored-by: Amp <amp@ampcode.com>
6 daysShow epic progress count instead of Epic label and checkboxBen Sima
Changed epic display in tree view from: t-PpXWsU [Epic] [ ] Task Manager Improvements To: t-PpXWsU [6/11] Task Manager Improvements The [6/11] shows completed/total child tasks, giving immediate visual feedback on epic progress. Regular tasks still use checkbox indicators: [ ] open, [~] in-progress, [✓] done.
6 daysMove namespace label between status and title in tree viewBen Sima
Changed tree output format from: t-abc123 [ ] Task title [Omni/Task.hs] To: t-abc123 [ ] [Omni/Task.hs] Task title This makes the namespace more prominent and groups all metadata (status + namespace) together before the title.
6 daysImprove task tree visualization displayBen Sima
Changes: 1. Remove [Task] label - only show [Epic] for epics, cleaner output 2. Truncate long titles to fit 80 columns with ... ellipsis 3. Better spacing with type label included in layout calculation Created task for future improvement: prettier box-drawing characters (├──, └──) which would require Data.Tree library investigation. Current output is clean and readable within standard terminal width.