Skip to content

Conversation

@jakejarvis
Copy link
Owner

@jakejarvis jakejarvis commented Oct 18, 2025

Summary by CodeRabbit

  • Chores
    • Rebranded to "Domainstack" across docs, README, UI copy, footer attribution, metadata, and package metadata.
    • Replaced artwork/logo SVG and updated watermark text and search placeholder to domainstack.io.
    • Switched persisted keys: theme -> "theme", history -> "search-history".
    • Updated environment placeholders (Postgres DATABASE_URL, INNGEST keys) and HOOT_USER_AGENT -> EXTERNAL_USER_AGENT.
    • Updated repository/links for issues and GitHub stars; adjusted Inngest id.

@vercel
Copy link

vercel bot commented Oct 18, 2025

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

Project Deployment Preview Comments Updated (UTC)
domainstack Canceled Canceled Oct 19, 2025 3:34pm
hoot Ready Ready Preview Comment Oct 19, 2025 3:34pm

💡 Enable Vercel Agent with $100 free credit for automated AI reviews

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 18, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Rebranding and configuration updates: project renamed from Hoot → Domainstack, environment variables adjusted (Postgres/Neon, Inngest keys, EXTERNAL_USER_AGENT), localStorage and theme keys renamed, GitHub repo references and package metadata updated, UI text/logo and docs changed. No control-flow or public API signature changes. (≤50 words)

Changes

Cohort / File(s) Summary
Environment & Config
\.env.example
Replaced Redis/Upstash creds with PostgreSQL (DATABASE_URL placeholder for Neon); added INNGEST_EVENT_KEY, INNGEST_SIGNING_KEY; renamed HOOT_USER_AGENTEXTERNAL_USER_AGENT.
Package & Repo Metadata
package.json
Project name, homepage and repository updated from hoot/hoot.sh to domainstack.io / jakejarvis/domainstack.io.
Documentation
README.md, .github/copilot-instructions.md
Full rebrand in docs and instructions: "Hoot" → "Domainstack", updated URLs and env var references.
App Metadata
app/layout.tsx
Updated exported metadata title string to "Domainstack — Domain Intelligence Made Easy".
UI Text & Attribution
components/app-footer.tsx, components/bookmarklet.tsx, components/domain/domain-search.tsx
Updated footer attribution (logo label, creator/name), bookmarklet description, and domain search placeholder to Domainstack branding.
Issue & GitHub References
components/create-issue-button.tsx, components/github-stars.tsx
Switched GitHub repo slug and API endpoint references from jakejarvis/hoot to jakejarvis/domainstack.io.
Local Storage & Theme Keys
components/domain/domain-suggestions.tsx, components/domain/domain-suggestions.test.tsx, hooks/use-domain-history.ts, components/theme-provider.tsx
Renamed localStorage keys: hoot-historysearch-history; theme storage key hoot-themetheme. Tests updated.
Constants & Image Utilities
lib/constants.ts, lib/image.ts, server/inngest/client.ts
USER_AGENT now reads EXTERNAL_USER_AGENT with fallback domainstack.io/0.1; watermark text changed to domainstack.io; Inngest client id set to "domainstack".
Logo
components/logo.tsx
Replaced SVG structure and path data (new viewBox and three path elements) to reflect new logo design.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested labels

enhancement

Poem

🐰 I hopped in with a tiny cheer,

Domainstack donned a new veneer.
Keys and logos found their place,
Docs and links now show their face.
🥕 A joyful hop — a brand-new trace!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "Rename project from Hoot to Domainstack" directly and accurately reflects the primary objective of the changeset. The raw summary shows that all file changes—spanning environment variables, configuration files, documentation, UI components, storage keys, package metadata, branding references, and even the logo—are unified around a single rebranding effort from "Hoot" to "Domainstack." The title is concise, specific, and clear enough that a teammate reviewing the Git history would immediately understand the PR's intent without needing to examine individual files. This is not a vague or overly broad title; it precisely captures the main change the developer intended.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 971c9af and f894148.

📒 Files selected for processing (1)
  • components/logo.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

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

@codecov
Copy link

codecov bot commented Oct 18, 2025

Codecov Report

❌ Patch coverage is 26.66667% with 22 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.93%. Comparing base (1a94370) to head (971c9af).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
components/github-stars.tsx 0.00% 12 Missing ⚠️
components/logo.tsx 0.00% 4 Missing ⚠️
components/app-footer.tsx 0.00% 2 Missing ⚠️
hooks/use-domain-history.ts 0.00% 2 Missing ⚠️
app/layout.tsx 0.00% 1 Missing ⚠️
components/theme-provider.tsx 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
+ Coverage   70.87%   70.93%   +0.05%     
==========================================
  Files         139      139              
  Lines       10192    10217      +25     
  Branches      923      928       +5     
==========================================
+ Hits         7224     7247      +23     
- Misses       2933     2935       +2     
  Partials       35       35              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jakejarvis jakejarvis changed the title Rename project from Hoot to DomainStack Rename project from Hoot to Domainstack Oct 18, 2025
cursoragent and others added 2 commits October 18, 2025 20:05
Removed unnecessary attributes from the SVG element and updated path elements to include the 'currentColor' fill.
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a0abb13 and 86cadac.

⛔ Files ignored due to path filters (2)
  • app/favicon.ico is excluded by !**/*.ico
  • app/icon.svg is excluded by !**/*.svg
📒 Files selected for processing (1)
  • components/logo.tsx (1 hunks)
🧰 Additional context used
📓 Path-based instructions (5)
components/**/*.{ts,tsx}

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Components live in components/ with kebab-case filenames and export names in PascalCase

Files:

  • components/logo.tsx
**/*.{ts,tsx}

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

**/*.{ts,tsx}: Use TypeScript only with strict mode enabled
Prefer small, pure modules (approximately ≤300 LOC)
Use 2-space indentation across the codebase
Consolidate internal imports using @/... path aliases

**/*.{ts,tsx}: Use @/... path aliases for internal imports
TypeScript-only codebase with strict type-checking; prefer small, pure modules (~≤300 LOC)
Use 2-space indentation
Files and folders should use kebab-case names

Files:

  • components/logo.tsx
{app,components,hooks,lib,server}/**

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Use kebab-case for files and folders

Files:

  • components/logo.tsx
components/**/*.{tsx,ts}

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Client components must begin with "use client" when needed

Files:

  • components/logo.tsx
components/**/*.tsx

📄 CodeRabbit inference engine (AGENTS.md)

components/**/*.tsx: components/: reusable UI primitives with kebab-case filenames and PascalCase exports
Component exports should be PascalCase

Files:

  • components/logo.tsx
🪛 GitHub Actions: Run tests and upload coverage
components/logo.tsx

[error] 12-27: Biome formatting check failed. The formatter would have printed different content for this file (formatting differences detected). Run 'biome check --fix' or format components/logo.tsx to apply changes.

Updated SVG icon with new design elements and structure.
Updated SVG icon to include dark mode styling.
@jakejarvis jakejarvis merged commit 9eb4922 into main Oct 19, 2025
1 of 2 checks passed
@jakejarvis jakejarvis deleted the rebrand branch October 19, 2025 15:32
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.

3 participants