Skip to content

Add ThatDeveloperGuy network (thatdevpro, thatdeveloperguy, thataiguy)#1084

Open
Janady13 wants to merge 1 commit into
thedaviddias:mainfrom
Janady13:add-thatdevpro-network
Open

Add ThatDeveloperGuy network (thatdevpro, thatdeveloperguy, thataiguy)#1084
Janady13 wants to merge 1 commit into
thedaviddias:mainfrom
Janady13:add-thatdevpro-network

Conversation

@Janady13
Copy link
Copy Markdown

@Janady13 Janady13 commented May 24, 2026

Adds 3 entries from the ThatDeveloperGuy network — all sites publish spec-compliant llms.txt files at the documented URLs.

  • ThatDeveloperGuy — SDVOSB veteran-owned parent agency, founded 2017
  • ThatDevPro — production studio brand, 14-tier Engine Optimization stack
  • ThatAIGuy — AI engineering services arm

The network also publishes the aio-surfaces open-source toolkit (MIT) which generates llms.txt + aeo.json + entity.json + brand.json from a single site config — useful tooling for anyone implementing the llms.txt spec.

Studio: thatdevpro.com · Maintainer: Joseph W. Anady (ORCID 0009-0008-8625-949X).

Summary by CodeRabbit

  • New Features
    • Added three new website entries to the collection: ThatAIGuy, ThatDeveloperGuy, and ThatDevPro, with associated metadata including domain, description, category, and favicon information.

Review Change Stack

Adds:
- ThatDeveloperGuy (parent agency, founded 2017)
- ThatDevPro (production studio brand)
- ThatAIGuy (AI engineering arm)

All three sites ship spec-compliant llms.txt files. Studio publishes
the open-source aio-surfaces toolkit (MIT) at github.com/Janady13/aio-surfaces.
@Janady13 Janady13 requested a review from thedaviddias as a code owner May 24, 2026 03:14
@vercel
Copy link
Copy Markdown

vercel Bot commented May 24, 2026

Someone is attempting to deploy a commit to the David Dias Digital's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the generated:websites-json Touches generated data/websites.json. label May 24, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 24, 2026

Walkthrough

Updated data/websites.json with three new website entries—ThatAIGuy, ThatDeveloperGuy, and ThatDevPro—each containing domain, description, llmsTxtUrl, category, favicon, and publishedAt fields. The Téléassistance Sénior entry was relocated to a later position in the array, and the JSON structure was adjusted accordingly.

Changes

Website Data Updates

Layer / File(s) Summary
Add three new website entries
data/websites.json
Three new website objects for ThatAIGuy, ThatDeveloperGuy, and ThatDevPro are inserted with complete metadata including domain, description, llmsTxtUrl, category, favicon, and publishedAt.
Relocate Téléassistance Sénior entry
data/websites.json
Téléassistance Sénior is removed from its original position and re-inserted later in the websites array, reordering its placement within the registry.
Adjust JSON array closing
data/websites.json
The closing bracket for the top-level JSON array is repositioned to maintain valid JSON syntax after structural changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

area:content, automerge:candidate, lane:mdx-fast, risk:low

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description provides context about the three new entries and the ThatDeveloperGuy network, but does not acknowledge or follow the required template note about data/websites.json being auto-generated. Add a note acknowledging that data/websites.json is auto-generated and that changes should be made through .mdx files in packages/content/data/websites/ instead.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding three new website entries from the ThatDeveloperGuy network (ThatAIGuy, ThatDeveloperGuy, ThatDevPro).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Review ran into problems

🔥 Problems

These MCP integrations need to be re-authenticated in the Integrations settings: Linear


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.

Warning

⚠️ This pull request might be slop. It has been flagged by CodeRabbit slop detection and should be reviewed carefully.

Copy link
Copy Markdown

@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.

🧹 Nitpick comments (1)
data/websites.json (1)

5409-5417: ⚡ Quick win

Keep this entry in generator-consistent sorted order.

"Téléassistance Sénior" appears manually relocated out of the name-sorted flow (see Line 5409 onward). Since scripts/generate-websites.ts sorts by name before writing data/websites.json, this placement is likely to cause unnecessary diff churn on the next regeneration.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@data/websites.json` around lines 5409 - 5417, The entry for "Téléassistance
Sénior" is out of the name-sorted order and will be shuffled by
scripts/generate-websites.ts on regeneration; move this JSON object so its
"name" field fits alphabetically among the other entries (maintaining the full
object exactly as shown, including "llmsTxtUrl", "category", "favicon", and
"publishedAt") to restore generator-consistent ordering and avoid unnecessary
diffs.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@data/websites.json`:
- Around line 5409-5417: The entry for "Téléassistance Sénior" is out of the
name-sorted order and will be shuffled by scripts/generate-websites.ts on
regeneration; move this JSON object so its "name" field fits alphabetically
among the other entries (maintaining the full object exactly as shown, including
"llmsTxtUrl", "category", "favicon", and "publishedAt") to restore
generator-consistent ordering and avoid unnecessary diffs.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a0f93b08-3e7a-4d1c-ae6a-0d036b5b0b8f

📥 Commits

Reviewing files that changed from the base of the PR and between e0f4bf9 and 6ead579.

📒 Files selected for processing (1)
  • data/websites.json

Copy link
Copy Markdown

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

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: 6ead5792fe

ℹ️ 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 data/websites.json
Comment on lines +5138 to +5142
"name": "ThatAIGuy",
"domain": "https://thataiguy.org",
"description": "AI engineering services from the ThatDeveloperGuy network. Custom cognitive architectures, federated AI systems, AI search engine integration, RAG and agent automation.",
"llmsTxtUrl": "https://thataiguy.org/llms.txt",
"category": "ai-ml",
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 Add matching MDX sources for new websites

Do not add entries directly to this file: data/websites.json is generated from packages/content/data/websites/*.mdx (scripts/generate-websites.ts) and is periodically regenerated by .github/workflows/update-websites-json.yml. Since this commit only edits the generated JSON and adds no corresponding MDX files, these new sites will be dropped on the next regeneration and won’t propagate to other MDX-derived artifacts, so the change is not durable.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

generated:websites-json Touches generated data/websites.json.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant