Trim Step 1 (build ontology) prompts to need-to-know#62
Merged
Conversation
Drop the file-by-file enumerations from Step 1 prompts — the agent + /rai-build-starter-ontology can introspect data/ and pick concept shapes itself. Keep only the modeling hints that are non-obvious from CSV inspection alone: - telco: keep the composite-key time-series hint (date+region) - energy: bare trim (every CSV maps cleanly to a Concept) - supply_chain: bare trim - machine_maintenance: keep the Period-concept hint (synthesized from an integer column, not its own CSV) - portfolio: keep the binary-covariance hint (long-form covariance is a binary property on Stock, not a separate Concept) + the Sector promotion hint (so downstream rules can aggregate per sector) Verified by per-template subagents that the tighter prompt + skill + data still lands on the canonical concept set.
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.
Why
The Step 1 prompts in each template's
runbook.mdenumerated every CSV indata/(e.g. "…from the eight CSVs in data/: cell_towers, network_equipment, …"). That's noise —/rai-build-starter-ontologyintrospectsdata/and picks concept shapes itself. The prompts read more like an instruction manual than what a user would type.This PR trims them to need-to-know only: the destination (
data/) and any modeling hint that's non-obvious from CSV inspection alone.Changes
telco_network_recoveryenergy_grid_planningsupply_chain_resiliencemachine_maintenancePeriodConcept (1..N fromperiodcolumn)Periodis synthesized from an integer column, not its own CSVportfolio_balancing(stock_i, stock_j, value)) + promoteSectorto a ConceptTest plan
rai-build-starter-ontologyskill, inspected CSV headers, and wrote out the PyRel ontology code an agent would produce from the trimmed prompt. All five concept sets match canonical.Conclusion: trims are safe. The agent generates equivalent loading code from either prompt form; the trim only removes redundant detail from the user's question.