Skip to content

fix: stabilize public production site - #3

Open
DjNastor wants to merge 14 commits into
mainfrom
fix/production-site-baseline
Open

fix: stabilize public production site#3
DjNastor wants to merge 14 commits into
mainfrom
fix/production-site-baseline

Conversation

@DjNastor

Copy link
Copy Markdown
Owner

Aligns package metadata and Vercel with Vinext/Vite and Node 24, adds CI, SEO metadata, robots, sitemap, and public deployment documentation.

@vercel

vercel Bot commented Jul 29, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
lukulu-recordings Error Error Jul 29, 2026 10:31am

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 36b13047cd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/robots.ts Outdated
@@ -0,0 +1 @@
import type { MetadataRoute } from "next";\n\nexport default function robots(): MetadataRoute.Robots {\n return { rules: [{ userAgent: "*", allow: "/" }], sitemap: "https://lukulurecordings.com/sitemap.xml" };\n}\n No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Write the robots route with actual newlines

The file contains literal \n byte sequences rather than line breaks. When Node 24 or the Vinext/Vite production build parses this route, the first backslash after the import is rejected as an invalid TypeScript token (Expected unicode escape), so the application cannot build or deploy until these escapes are replaced with real newlines.

Useful? React with 👍 / 👎.

Comment thread app/sitemap.ts Outdated
@@ -0,0 +1 @@
import type { MetadataRoute } from "next";\n\nexport default function sitemap(): MetadataRoute.Sitemap {\n return [{ url: "https://lukulurecordings.com/", lastModified: new Date(), changeFrequency: "weekly", priority: 1 }, { url: "https://lukulurecordings.com/labels", lastModified: new Date(), changeFrequency: "monthly", priority: 0.8 }];\n}\n No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Write the sitemap route with actual newlines

This source is likewise stored as one line containing literal \n sequences. Any production build that discovers the sitemap route will fail while parsing the backslash immediately after the import, preventing deployment even if the robots route is corrected separately; replace the escapes with real line breaks.

Useful? React with 👍 / 👎.

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.

1 participant