WIP: Feat/daydream implementation - #14
Open
vinkent420 wants to merge 27 commits into
Open
vinkent420 wants to merge 27 commits into
vinkent420 wants to merge 27 commits into
Conversation
Keep Daydream as the mandatory creative-thesis step, retire the uncalibrated predictive Judge, and learn from exact downstream outcomes while preserving the latest shop, budget, CAD, and Pearlturn work from main.
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.
Pull request
Outcome
Make Daydream the mandatory first creative step of every
workshop start <inventor>cycle and turn it into a world-informed, Taste-governed creative product thesis, not an ordinary idea generator or another product-run stage.Each Dream now:
The predictive Daydream Judge is removed from the active runtime. Replay against real outcomes showed that it rejected both failed builds and published products, making it an uncalibrated admission wall. Its useful nine dimensions remain as an Inventor pre-commit audit inside the single Daydream Goal. Historical verdicts remain readable but no longer gate Wish creation.
Ownership
daydreamworkflow,cli,repository@deehwworkflowandrepositorycontracts are critical;daydreamandcliare standard)Change class
Current and resulting behavior
Before:
After:
workshop start; directworkshop wishremains the separate path for an already-authored human Wish.Explicitly out of scope:
Boundaries and contracts
Public contract added or changed:
workshop.daydreamschema-v2/v3 thesis models and portable validation.Opportunity,WorldScan,WorldSignal,Experience,ProofPlan, andLearningTrace.DaydreamProvenanceand exact context identities.RunOutcomeMemory,remember_run_outcome, andremember_resumed_outcome.Components consuming it:
clistarts or reloads a sealed Dream and records run outcomes.workflowemits checkpoint-verified lineage in native-run receipts.repositorypackaging includes the exact prior-work catalog.Dependency direction:
workshop.daydreamandworkshop.workflowAPIs.External ports or effects:
Durable formats read or written:
work/IDEA.json: schema versions 1–3; new writer uses v3.PROVENANCE.json: schema v1.NOTEBOOK.jsonl: historical versions retained; new learning records use v3.OUTCOMES.jsonl: versions 1–2; new writer uses v2.lineage: schema v1.src/workshop/daydream/prior_work.json.The change stays in its owning source and mirrored test directories.
Components import public contracts, not sibling private implementations.
Workflow remains the only stage sequencer.
Product work remains one persistent native session per Wish and one active Codex Goal per cognitive stage attempt; Python does not own the improvement loop.
CLI and inventor code depend on Workshop; Workshop does not import them.
No new
core,foundation,common, orutilsdumping ground was added.Conditional review
Inventor
TASTE.mdremains canonical and workflow-bound..codex/agents/.Skill or dependency
Schema, artifact, or durable state
dream-againverdicts no longer prevent build admission.Outside or physical effect
Verification
git diff --checkCompatibility and release note
changes/daydream-mvp.added.mdReviewer guide
Start with these files:
src/workshop/daydream/prompt.pysrc/workshop/daydream/contracts.pyandsrc/workshop/daydream/schema.pysrc/workshop/daydream/native.pysrc/workshop/daydream/outcomes.pyandsrc/workshop/workflow/native_run.pytests/end_to_end/test_daydream_learning_loop.pyPrimary tradeoff: removing the predictive Judge may allow more hypotheses to consume build time, but avoids a second uncalibrated native session that produced false negatives on already-published products. Quality control remains split honestly: Daydream performs rigorous pre-commit falsification; Invent/Make/Playtest/Release provide the evidence; later Dreams learn from those exact outcomes.