Skip to content

Conversation

@bbenligiray
Copy link
Member

@bbenligiray bbenligiray commented Aug 25, 2025

@bbenligiray bbenligiray self-assigned this Aug 25, 2025
const fileContent = fs.readFileSync(indexPath, 'utf-8');
const match = fileContent.match(/---\s*title:\s*(.*?)\s*---/);
const title = match ? match[1] : path.basename(file);
const url = `https://docs.api3.org${file}`;
Copy link
Member Author

Choose a reason for hiding this comment

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

I realized that the links in the hinter net docs weren't full

@github-actions
Copy link

github-actions bot commented Aug 25, 2025

Visit the preview URL for this PR (updated for commit 4fc15bd):

https://oev-docs--pr222-llms-evy8886b.web.app

(expires Tue, 02 Sep 2025 13:52:19 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 6915b094b5ba83fde754632ba50c1ee9406d433f

Copy link
Collaborator

@dcroote dcroote left a comment

Choose a reason for hiding this comment

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

Cool. One big issue though: all of the llms.txt links are broken because they use the .md extension rather than the built / published .html.

Also, I feel this is motivation to improve some titles. For example, llms-full.txt has one topline header as # Quickstart. Quickstart what? Since the page hierarchy isn't well captured in llms-full.txt, maybe "dApps Quickstart"? etc.

@bbenligiray
Copy link
Member Author

All very good points, thanks

@bbenligiray bbenligiray requested a review from dcroote August 26, 2025 13:52
Copy link
Collaborator

@dcroote dcroote left a comment

Choose a reason for hiding this comment

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

Nice 👍 and clever on the header text clues

Copy link
Collaborator

@Siegrift Siegrift left a comment

Choose a reason for hiding this comment

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

👍 LGTM (I only browsed the generated content for LLM and it looks pretty good).

@bbenligiray bbenligiray merged commit 42198b5 into main Aug 26, 2025
3 of 4 checks passed
@bbenligiray bbenligiray deleted the llms branch August 26, 2025 14:25
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.

Implement an easy way to copy the entire content for use with LLMs

4 participants