-
Notifications
You must be signed in to change notification settings - Fork 106
build: generate llms.txt and markdown files of the documentation for optimal AI consumption #2109
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
f3a2a4b
to
cf3222c
Compare
b551e1e
to
6acc6d9
Compare
06ef1c1
to
0857ffd
Compare
0857ffd
to
e66ad32
Compare
ef5254e
to
e2bcf25
Compare
e2bcf25
to
630f62a
Compare
buildDir + 'markdown-and-sources-data.json', | ||
{ | ||
outputFilePath: path.join(parentOfDocs, 'llms.txt'), | ||
baseUrl: 'https://instructure.design/pr-preview/pr-2109/markdowns/', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is for testing purposes, it should be 'https://instructure.design/markdowns/'
f57ff58
to
cc9f407
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice work!! There are a couple of smaller issues what needs to be resolved (see my comments)
Also we should think about (its a future ticket) what we do with internal links. Now they are anchor link,s AI might have a hard time following them in the .md
files..
packages/__docs__/buildScripts/ai-accessible-documentation/generate-ai-accessible-markdowns.mts
Show resolved
Hide resolved
packages/__docs__/buildScripts/ai-accessible-documentation/generate-ai-accessible-markdowns.mts
Outdated
Show resolved
Hide resolved
packages/__docs__/buildScripts/ai-accessible-documentation/generate-ai-accessible-markdowns.mts
Outdated
Show resolved
Hide resolved
packages/__docs__/buildScripts/ai-accessible-documentation/generate-ai-accessible-markdowns.mts
Outdated
Show resolved
Hide resolved
packages/__docs__/buildScripts/ai-accessible-documentation/generate-ai-accessible-llms-file.mts
Outdated
Show resolved
Hide resolved
packages/__docs__/buildScripts/ai-accessible-documentation/summaries-for-llms-file.json
Outdated
Show resolved
Hide resolved
packages/__docs__/buildScripts/ai-accessible-documentation/generate-ai-accessible-markdowns.mts
Outdated
Show resolved
Hide resolved
packages/__docs__/buildScripts/ai-accessible-documentation/generate-ai-accessible-markdowns.mts
Show resolved
Hide resolved
cc9f407
to
e66ad32
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is one small change needed for type parsing, see my comment. Also dont forget to use the proper link here: https://github.com/instructure/instructure-ui/pull/2109/files#diff-1faeaf1714eeac41ba8eccaff204c2f140f9adc2d5e6964a357723ba7ccde096R163
f00ceab
to
e1632c5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work, seems to be working fine. I'm not sure if the AI components section will confuse the llms or not, but I don't think it should be added as an exeption.
Approved
e1632c5
to
91ea593
Compare
…optimal AI consumption INSTUI-4701
91ea593
to
a508506
Compare
INSTUI-4701
ISSUE:
TEST PLAN: