chore(auto-carousel): remove API route, form component, page, and all references#87
Open
natashaannn wants to merge 4 commits into
Open
chore(auto-carousel): remove API route, form component, page, and all references#87natashaannn wants to merge 4 commits into
natashaannn wants to merge 4 commits into
Conversation
Removes all files that constitute the auto-carousel feature: - app/api/auto-carousel/route.ts — Claude API + CaptionExtractor backend - app/components/AutoCarouselForm.tsx — the pro-gated form UI - app/auto-carousel/page.tsx — the route page (Issue #76 scope, pulled forward because AutoCarouselForm was its only meaningful dependency and now gone) Deleting the page here rather than leaving a broken import unresolvable by tsc. AC: POST /api/auto-carousel now returns 404; no AutoCarouselForm import anywhere
…d login Cleans up every remaining mention of /auto-carousel or AutoCarousel in production code: - app/page.tsx: removes the Auto Bulk Carousel Pro tool card and related imports (IconWand, IconLock, Box); consolidates the philosophy section text - app/components/Header.tsx: removes the /auto-carousel nav entry and unused IconWand import from NAV_LINKS - app/login/page.tsx: redirects post-login to /carousel instead of the deleted /auto-carousel route; removes feature-specific copy Also adds smoke render tests for Header and the landing page asserting that no link to /auto-carousel exists in either component. AC: #1 (zero grep hits), #3 (no tool card on landing page), #4 (no nav link)
…Files The file no longer exists following issue #81 cleanup.
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.
Summary
app/api/auto-carousel/route.tsandapp/components/AutoCarouselForm.tsx— the Claude-backed LLM route and its form component are fully removed;POST /api/auto-carouselnow returns 404./auto-carouselreferences fromapp/page.tsx(tool card),app/components/Header.tsx(nav link), andapp/login/page.tsx(post-login redirect now points to/carousel).app/auto-carousel/page.tsxas well — Issue refactor(app): reorganise routes into (public) and (internal) Next.js route groups #76 (which was supposed to handle this) never merged, so the cleanup is completed here.CLAUDE.mdto remove the now-deletedAutoCarouselForm.tsxfrom the Key Source Files table.How to review
app/api/auto-carousel/route.ts— entire file deleted; verify no import of this route remains.app/components/AutoCarouselForm.tsx— entire file deleted;app/page.tsxdiff should show only its import removal and the tool card block.app/page.tsx— should lose exactly the Auto Bulk Carousel card and its import; two remaining cards now fill/center the container.app/components/Header.tsx— diff should show only the removal of the Auto Carousel nav entry.app/login/page.tsx— redirect target changed from/auto-carousel→/carousel; text updated. This file will be deleted by Issue refactor(app): reorganise routes into (public) and (internal) Next.js route groups #76.app/components/Header.test.tsx+app/page.test.tsx— new tests asserting no/auto-carousellinks remain in the rendered header and landing page.Test plan
npm testpasses (369 tests, node + react projects)npm run buildsucceeds — no/auto-carouselroute in route manifesttsc --noEmitpasses — no dangling importsManual verification completed:
/) — no Auto Bulk Carousel card, no/auto-carousellink; remaining two cards fill and center the containerPOST /api/auto-carouselreturns 404Closes #81
🤖 Generated with Claude Code