Skip to content

Conversation

@thedaviddias
Copy link
Owner

@thedaviddias thedaviddias commented Sep 18, 2025

This PR adds helloskin.hu to the llms.txt hub.

Submitted by: kkinyo

Website: https://helloskin.hu/
llms.txt: https://helloskin.hu/llms.txt
llms-full.txt: https://helloskin.hu/llms-full.txt
Category: ecommerce-retail


This PR was created via admin token for a user without GitHub repository access.

Please review and merge if appropriate.

Summary by CodeRabbit

  • New Features
    • Added a new website entry: helloskin.hu.
    • Includes Hungarian description, category, and publication date.
    • Provides links to LLMs access files (llms.txt and llms-full.txt) for the domain.
    • Visible as a new item in the websites dataset/listing; no functional changes elsewhere.

@vercel
Copy link

vercel bot commented Sep 18, 2025

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

Project Deployment Preview Comments Updated (UTC)
llms-txt-hub Ready Ready Preview Comment Sep 18, 2025 5:19pm

@coderabbitai
Copy link

coderabbitai bot commented Sep 18, 2025

Walkthrough

Adds a new MDX data entry for helloskin.hu under packages/content/data/websites. The file defines front matter fields (name, description, website, llmsUrl, llmsFullUrl, category, publishedAt) and a body with a header and Hungarian description mirroring the front matter. No code logic changes.

Changes

Cohort / File(s) Summary
Content data addition
packages/content/data/websites/helloskin-hu-llms-txt.mdx
New MDX dataset entry for helloskin.hu with front matter (name, description, website, llmsUrl, llmsFullUrl, category, publishedAt) and matching body content; references llms.txt and llms-full.txt URLs.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

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 title "feat: add helloskin.hu to llms.txt hub" is a concise, single-sentence summary that directly describes the primary change (adding helloskin.hu to the llms.txt hub) and aligns with the added MDX dataset file and PR description; it is specific and free of vague language or extraneous noise.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch submit-helloskin-hu-1758215799005

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.


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.

Copy link

@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

🧹 Nitpick comments (1)
packages/content/data/websites/helloskin-hu-llms-txt.mdx (1)

11-13: Optional: Add a brief English summary for discoverability.

Keeping the Hungarian tagline is great; consider adding one English sentence in the body to aid non‑HU readers/search.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between a9b5ed0 and ff02401.

📒 Files selected for processing (1)
  • packages/content/data/websites/helloskin-hu-llms-txt.mdx (1 hunks)
🧰 Additional context used
🔍 Remote MCP Linear

Additional context (from Linear)

  • Related llms.txt hub issues: THE-1133 “Add vercel llms.txt” — Done; THE-1125 “https://llmstxthub.com/llms.txt is returning a 500” — Done; THE-1112 “Add email to subscribe does not work” — Done; THE-1134 “Add upsun.com” — Todo; THE-1123 “The latest news are not updating” — Backlog.

  • Linear docs include "Pull Request Reviews" guidance (reviews handled via GitHub).

  • No "contributing" documents returned from Linear document search.

  • Search for user "thedaviddias" returned no Linear user results.

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Validate
🔇 Additional comments (4)
packages/content/data/websites/helloskin-hu-llms-txt.mdx (4)

1-9: Front matter looks complete and consistent.

All required fields appear present and well‑formed. Nice.


8-8: Date format sanity check.

publishedAt: '2025-09-18' is ISO‑8601 (YYYY‑MM‑DD). Confirm this aligns with the site’s build expectations (no timezone component needed).


7-7: Category validated — 'ecommerce-retail' is an allowed category.
packages/content/data/websites/helloskin-hu-llms-txt.mdx (line 7): no change required.


1-9: No duplicates found — safe to merge. Search of packages/content/data/websites (excluding packages/content/data/websites/helloskin-hu-llms-txt.mdx) returned no other occurrences of "helloskin.hu".

Comment on lines +5 to +6
llmsUrl: 'https://helloskin.hu/llms.txt'
llmsFullUrl: 'https://helloskin.hu/llms-full.txt'
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue

🧩 Analysis chain

Verify llms.txt endpoints are reachable and non‑empty.

Please confirm both URLs return 200, text/plain (or similar), and non‑empty content.

Run:


🏁 Script executed:

#!/bin/bash
set -euo pipefail

check() {
  url="$1"
  echo "==> $url"
  # Show final status and content-type
  curl -sI -L "$url" | awk 'BEGIN{code="";ct=""} /^HTTP/{code=$2} tolower($0) ~ /^content-type:/{ct=$0} END{print "STATUS:",code; print ct}'
  # Fetch a small sample and count bytes
  bytes=$(curl -sL "$url" | head -c 2048 | wc -c | awk '{print $1}')
  echo "SAMPLE_BYTES:" "$bytes"
  test "$bytes" -gt 0 || { echo "ERROR: empty content"; exit 1; }
  echo
}

check "https://helloskin.hu/llms.txt"
check "https://helloskin.hu/llms-full.txt"

Length of output: 195


Fix broken llmsFullUrl (returns 404 / HTML)

packages/content/data/websites/helloskin-hu-llms-txt.mdx (lines 5–6)

  • llmsUrl: OK — 200, content-type: text/plain;charset=UTF-8, sample bytes: 2048.
  • llmsFullUrl: FAIL — 404, content-type: text/html; charset=UTF-8. Update the URL to a reachable text/plain resource or remove the llmsFullUrl reference.
🤖 Prompt for AI Agents
In packages/content/data/websites/helloskin-hu-llms-txt.mdx around lines 5 to 6,
the llmsFullUrl currently points to 'https://helloskin.hu/llms-full.txt' which
returns 404 HTML; either replace llmsFullUrl with a valid reachable text/plain
URL that serves the full LLMS data (ensure it returns 200 and text/plain) or
remove the llmsFullUrl field entirely from the file if no replacement is
available; update any consumers/docs accordingly.

@thedaviddias thedaviddias merged commit 6d40e14 into main Oct 16, 2025
4 of 5 checks passed
@thedaviddias thedaviddias deleted the submit-helloskin-hu-1758215799005 branch October 16, 2025 16:13
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