feat: add Digital Inning to llms.txt hub#389
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
WalkthroughAdded a new MDX content file defining a website entry for “Digital Inning” with frontmatter fields (name, description, website, llmsUrl, llmsFullUrl, category, publishedAt) and a simple body (H1 and paragraph mirroring the description). Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
✨ Finishing Touches🧪 Generate unit tests
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (4)
packages/content/data/websites/digital-inning-llms-txt.mdx (4)
5-6: Prefer concise llms.txt and move long content to llms-full.txt.llms.txt is very large (~1500 lines). Keep llms.txt minimal and put verbose sections in llms-full.txt once it’s correctly hosted as text/plain.
3-3: Tone: avoid superlatives; use neutral, directory-style copy.Suggested copy to keep it factual and consistent with a listings hub:
-description: 'Discover top-tier digital marketing services in India with Digital Inning, the leading agency based in Rajkot. Elevate your online presence today.' +description: 'Digital Inning is a digital marketing agency based in Rajkot, India, offering services such as SEO, PPC, social media, content, and website development.' ... -Discover top-tier digital marketing services in India with Digital Inning, the leading agency based in Rajkot. Elevate your online presence today. +Digital Inning is a digital marketing agency in Rajkot, India, providing SEO, PPC, social media, content, and website development services.Also applies to: 13-13
11-11: H1 is fine; optionally omit body duplication.The H1 plus repeated description is acceptable, but you can omit the body paragraph to avoid duplication—frontmatter description is already surfaced in listings.
1-1: Minor: ensure trailing newline at EOF.Some linters flag missing newline at end of file.
Also applies to: 14-14
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
packages/content/data/websites/digital-inning-llms-txt.mdx(1 hunks)
⏰ 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 (2)
packages/content/data/websites/digital-inning-llms-txt.mdx (2)
2-9: Frontmatter fields look correct.name, description, website, llmsUrl, llmsFullUrl, category, and publishedAt align with typical entries; date format is ISO-8601.
7-7: No change needed: ‘agency-services’ is a valid category. Found in agile-it-cmmc-compliance-llms-txt.mdx, yellowlime-llms-txt.mdx, and digital-inning-llms-txt.mdx, confirming it’s already part of the site taxonomy.
| llmsUrl: 'https://digitalinning.com/llms.txt' | ||
| llmsFullUrl: 'https://digitalinning.com/llms-full.txt' |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
Verify target URLs resolve and are correctly typed.
- llms.txt is reachable and returns plaintext, but appears to contain full marketing content. (digitalinning.com)
- llms-full.txt currently returns HTML (likely a redirect/404 fallback), not a plaintext file. Please confirm the file exists at the given path or update the URL. (digitalinning.com)
Run to verify status and content types:
🏁 Script executed:
#!/bin/bash
set -euo pipefail
for u in \
'https://digitalinning.com/llms.txt' \
'https://digitalinning.com/llms-full.txt'
do
echo "==> $u"
curl -sIL "$u" | sed -n '1,15p'
echo "--- content-type ---"
curl -sI "$u" | grep -i '^content-type'
doneLength of output: 1694
Update LLMS URLs
llms.txt(https://digitalinning.com/llms.txt) returns HTTP 200 with content-typetext/plain—no change needed.llms-full.txt(https://digitalinning.com/llms-full.txt) returns HTTP 302 and HTML (redirects to homepage). In packages/content/data/websites/digital-inning-llms-txt.mdx (lines 5–6), correctllmsFullUrlto the actual plaintext endpoint or remove if unavailable.
🤖 Prompt for AI Agents
In packages/content/data/websites/digital-inning-llms-txt.mdx around lines 5–6,
the llmsFullUrl currently points to https://digitalinning.com/llms-full.txt
which returns an HTTP 302 redirect to HTML; update llmsFullUrl to the actual
plaintext endpoint if one exists (replace the URL with the direct .txt endpoint)
or remove the llmsFullUrl property entirely if no plaintext full list is
available; verify the new URL returns HTTP 200 with content-type text/plain
after making the change.
This PR adds Digital Inning to the llms.txt hub.
Submitted by: dhaval.whiznic
Website: https://digitalinning.com/
llms.txt: https://digitalinning.com/llms.txt
llms-full.txt: https://digitalinning.com/llms-full.txt
Category: agency-services
This PR was created via admin token for a user without GitHub repository access.
Please review and merge if appropriate.
Summary by CodeRabbit