Refactor features: Replace FeaturePage with FeatureSection

[markopiers]

Updated imports, and enhance component structure of feature section for improved maintainability

Summary by CodeRabbit

• Refactor

  • Removed a redundant Features wrapper and now uses the Features component directly.
  • Renamed several Features components for consistent naming (title, card, section).
  • Swapped to updated Hero and Feature front-page components while preserving render order and layout.

• Chores

  • Cleaned up unused exports related to the old Features wrapper.

No visual or functional changes; homepage content and order remain unchanged.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

Removes the features wrapper page at src/app/features/page.tsx, updates src/app/page.tsx to import and render HeroFrontPage and FeatureSection directly, and renames several feature component default exports and usages (FeaturePage→FeatureSection, ReusableFeatureCard→FeatureCard, FeatureTitleAndDescription→FeatureTitle).

Changes

Cohort / File(s)	Summary
Remove features wrapper page src/app/features/page.tsx	Deleted file that default-exported FeatureFrontPage, which previously returned FeaturePage.
Home page import/update src/app/page.tsx	Replaced imports/usages: HeroSection→HeroFrontPage (import ./hero/page) and FeatureFrontPage→FeatureSection (import @/components/Features/FeatureSection); render order unchanged.
Feature components rename src/components/Features/FeatureSection.tsx, src/components/Features/FeatureCard.tsx, src/components/Features/FeatureTitle.tsx	Renamed default exports and adjusted local usages/imports: FeaturePage→FeatureSection, ReusableFeatureCard→FeatureCard, FeatureTitleAndDescription→FeatureTitle.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    actor User
    participant App
    participant HomePage as "src/app/page.tsx"
    participant Hero as "HeroFrontPage"
    participant FeatureSection as "FeatureSection"
    participant FeatureTitle as "FeatureTitle"
    participant FeatureCard as "FeatureCard(s)"

    User->>App: GET /
    App->>HomePage: render
    HomePage->>Hero: render HeroFrontPage
    HomePage->>FeatureSection: render FeatureSection
    FeatureSection->>FeatureTitle: render title/description
    FeatureSection->>FeatureCard: render cards list
    FeatureSection-->>HomePage: section rendered
    HomePage-->>App: response ready

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• Refactor feature components #26: Modifies the feature component exports and imports (FeaturePage→FeatureSection and related renames); directly overlaps with these refactors.
• Add FeatureTitleAndDescription Component and Refactor FeaturePage Layout #10: Adds/removes the src/app/features/page.tsx wrapper that this PR deletes; touches the same exported entity.
• Refactor features: Replace FeaturePage with FeatureSection #28: Performs a similar refactor removing the wrapper and updating Home imports/usages to FeatureSection; strongly related.

Suggested reviewers

• markopiers

Poem

I nibbled old wrappers, hopped through the code,
Swapped names and paths down the tidy road.
Cards and titles now wear fresh fur,
Home page hops lighter—thump, little spur. 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title succinctly and accurately describes the primary change in the PR — a refactor replacing FeaturePage with FeatureSection — and aligns with the provided objectives and file-level summaries showing component renames and updated imports. It is clear, specific, and short enough for teammates scanning history to understand the main intent.

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c1836e0 and 5e47a85.

📒 Files selected for processing (1)

• src/app/page.tsx (1 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

src/components/Features/FeatureCard.tsx (1)

4-8: Update type name to match the component rename.

The type ReusableFeatureCardProps should be renamed to FeatureCardProps to maintain consistency with the component name change.

-type ReusableFeatureCardProps = {
+type FeatureCardProps = {
   title: string;
   description: string;
   icon: React.ElementType;
 };

And update the function parameter type:

 export default function FeatureCard({
   icon,
   title,
   description,
-}: ReusableFeatureCardProps) {
+}: FeatureCardProps) {

src/components/Features/FeatureSection.tsx (1)

12-16: Update component usage to match the corrected import.

The component is still being used as ReusableFeatureCard but should be FeatureCard to match the import fix above.

-            <ReusableFeatureCard
+            <FeatureCard
               title={card.title}
               description={card.description}
               icon={card.icon}
             />

🧹 Nitpick comments (1)

src/app/page.tsx (1)

6-6: Unify hero component naming (remove 'FrontPage' suffix)

Found a single "FrontPage" usage: src/app/hero/page.tsx exports default HeroFrontPage() { return } and src/app/page.tsx (line 6) imports it via ./hero/page. Recommendation: either import HeroPage directly in src/app/page.tsx or rename the wrapper to HeroPage to match other sections.

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f3abc43 and c1836e0.

📒 Files selected for processing (5)

• src/app/features/page.tsx (0 hunks)
• src/app/page.tsx (1 hunks)
• src/components/Features/FeatureCard.tsx (1 hunks)
• src/components/Features/FeatureSection.tsx (1 hunks)
• src/components/Features/FeatureTitle.tsx (1 hunks)

💤 Files with no reviewable changes (1)

• src/app/features/page.tsx

🧰 Additional context used

🧬 Code graph analysis (2)

src/components/Features/FeatureSection.tsx (1)

src/components/Features/FeatureTitle.tsx (1)

• FeatureTitle (1-18)

src/app/page.tsx (2)

src/app/hero/page.tsx (1)

• HeroFrontPage (3-7)

src/components/Features/FeatureSection.tsx (1)

• FeatureSection (5-22)

🔇 Additional comments (5)

src/components/Features/FeatureCard.tsx (1)

10-14: LGTM! Component rename implemented correctly.

The rename from ReusableFeatureCard to FeatureCard improves naming consistency and clarity. The function signature and implementation remain unchanged, maintaining backward compatibility with existing props.

src/components/Features/FeatureTitle.tsx (1)

1-1: LGTM! Clean component rename.

The rename from FeatureTitleAndDescription to the more concise FeatureTitle improves naming clarity while maintaining all functionality intact.

src/components/Features/FeatureSection.tsx (1)

5-8: LGTM! Component rename and updated import usage.

The rename from FeaturePage to FeatureSection and the usage of the updated FeatureTitle component are implemented correctly.

src/app/page.tsx (2)

5-5: LGTM! Clean import path update.

The import has been correctly updated to use the direct component path @/components/Features/FeatureSection instead of the removed wrapper page.

---

10-13: LGTM! Component usage updated correctly.

The usage has been properly updated to FeatureSection matching the renamed component, maintaining the same functionality while improving the architecture by removing the unnecessary wrapper.

[coderabbitai]

⚠️ Potential issue

Import name inconsistency detected.

Line 2 imports the component as ReusableFeatureCard but it should be imported as FeatureCard to match the updated export name from FeatureCard.tsx.

-import ReusableFeatureCard from "./FeatureCard";
+import FeatureCard from "./FeatureCard";

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	import ReusableFeatureCard from "./FeatureCard";
	import FeatureTitle from "./FeatureTitle";
	import FeatureCard from "./FeatureCard";
	import FeatureTitle from "./FeatureTitle";

🤖 Prompt for AI Agents

In src/components/Features/FeatureSection.tsx around lines 2 to 3 the import
uses the name ReusableFeatureCard but the component was renamed/exported as
FeatureCard; update the import to use FeatureCard (i.e., import FeatureCard from
"./FeatureCard") so the import name matches the exported component and then
update any local references if they still use ReusableFeatureCard.