Initial Pass at documents necessary for AI assisted programming#61
Closed
Zelos Zhu (zdz2101) wants to merge 18 commits into
Closed
Initial Pass at documents necessary for AI assisted programming#61Zelos Zhu (zdz2101) wants to merge 18 commits into
Zelos Zhu (zdz2101) wants to merge 18 commits into
Conversation
Contributor
Jeremy Wildfire (jwildfire)
left a comment
There was a problem hiding this comment.
I like this in general! Tons of food for thought. I've added a couple of high-level comments below:
- This is probably too much for a first step. I'd lean towards scaling back and adding a small handful of things to the scaffold and then iterating
- We need a big readme/contributor guide update to go along with this that clearly explains how the humans set this up and what their respsonibilities are.
- Not entirely sure the R files are needed, but I might be wrong. I think we should generally prefer command line api calls (especially the
ghgithub tool) and markdown templates so that we are language-agnostic as much as possible - I'll paste parts of my current AGENT.md as a comment for your consideration. There's also a PR over in gsm.core with a draft from a few months back that might be useful reference: Gilead-BioStats/gsm.core#95
Next steps:
- Overall goal: Let's clearly align on scope of the first iteration of our
gsm AI scaffold- The deliverable here is probably a "spec" or design document.
- Get feedback from other team members on the initial draft(s)
Author
Yea I suspect we need some sort of "skill graph" of .mds that tells the story of the relationship between the packages and why they're related to each other, basically turning this into mds and potentially citing where each package is involved
|
Author
|
Closing to keep repos clean and organized |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Overview
Working in the gsm ecosystem of packages, it can prove difficult attaching the appropriate and full extent of context when using coding assistants, and even if you make some .md's , it can easily "drift" from the purpose/north star of our gsm ecosystem of packages.
With the powers of GPT-5.3-Codex, I've prototyped a slew of appropriate
.mds to "synchronize" a context window across our ecosystem and built 2 functions that can rapidly improve/speed up initial prompting to copilot (the tool Gilead is already providing for us) of any gsm ecosystem issueUnder
inst/ai_templates, I'll just call them a family of docs that pretty much describe basic development practices, etc. And maybe they're not all entirely necessary but they were needed/artifacts of spawning this idea/framework. Specifically I want to highlightECOSYSTEM.mdwhere we can enrich with examples or thorough descriptions of each package across all repos and a templatedARCHITECTURE.mdwhich will be repo specific. I also added a new issue template denoted a6-CONTEXT_PACK.mdthat feels like the necessary context alongside the ai docs to one-shot prompt copilot, for a pretty good first pass at code generation. I think if we iterate on the ai docs and then this refined context pack issue template it'll get better:2 functions were built:
sync_gsm_standards()which ensures our issue templates/cicd/ai docs stay synced across repos, and an associated cicd check for this has been created as well viaai-template-drift-check.yamlto especially ensure agents/coding assistants don't go too haywirebuild_agent_prompt()will create the appropriate templated prompt to grab & read these ai docs and begin a first pass at issues, it is as simple asbuild_agent_prompt("gsm.qtl#78"), copy/paste into the copilot chat on VS CodePlease try it yourself!
copy/paste the prompt into copilot, and proceed to write more detailed documentation on other issues as copilot gets stuff done