-
Notifications
You must be signed in to change notification settings - Fork 13
VERA 2.0 Architecture #170
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
Open
luca-belli
wants to merge
26
commits into
feat/VERA_2.0
Choose a base branch
from
feat/architure.md
base: feat/VERA_2.0
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
7d132e5
isolate human-reviewed conversations
jgieringer 9ede7c5
elaborate on data used in preprint
jgieringer 85dd6c6
adding new architecture file
luca-belli 937e42c
adding new architecture file
luca-belli 29e3c49
new architecture for entry-point
luca-belli 3151059
Resolve PR #170 review feedback: config/CLI design, naming scheme, mi…
luca-belli 112a934
Harden migration plan: fix phase-ordering bugs, add stable-interfaces…
luca-belli c008c42
Fix 8 critical gaps in the migration plan found under review
luca-belli 2db14bc
Flag that generation regression testing needs a two-part approach
luca-belli ec4ecf6
Move the engine-testing caution from Phase 1 to Phase 5, where it bel…
luca-belli 4293fa3
Fix 5 more gaps: dropped script, downstream naming break, manifest/co…
luca-belli 7a68de3
Rename packages to match subcommand names; remove stale planning docs
luca-belli 57d1659
fix: resolve architecture-doc inconsistencies and stale CODEOWNERS en…
luca-belli d29f6c4
fix: resolve architecture-doc inconsistencies and stale CODEOWNERS en…
luca-belli 8d2f4e5
Add storage abstraction (Phase S, orthogonal to the rest of the migra…
luca-belli 8954957
Propagate spine hardening: sha canonicalization, --target shorthand,
luca-belli 0235289
Add ARCHITECTURE-SPINE.md: terse, AD-numbered invariants contract
luca-belli 4e03d63
Mark ARCHITECTURE-SPINE.md as final
luca-belli 80116f1
Add Phase O: firm up OpenSpec adoption from a maybe into an orthogona…
luca-belli 7ea94c0
Resolve remaining PR #170 review threads: chatbot flag, path resoluti…
luca-belli 096014a
Sync architecture spine with today's PR #170 review resolutions
luca-belli b0e91ea
Add best-guess caveat to architecture.md and the spine
luca-belli b0d2502
Add statistical semantic-similarity tier to Phase 5 testing
luca-belli 6205236
Merge pull request #159 from SpringCare/mv-human-validated-assets
luca-belli 5a26e8c
Rename generation.models to generation.user to remove field-name ambi…
luca-belli e40e464
Merge remote-tracking branch 'origin/main' into feat/architure.md
luca-belli File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| # Architecture and domain packages require maintainer review. | ||
| /docs/architecture.md @SpringCare/vera-mh-maintainers | ||
| /docs/vera-cli-use-cases.md @SpringCare/vera-mh-maintainers | ||
| /llm_clients/ @SpringCare/vera-mh-maintainers | ||
| /judge/ @SpringCare/vera-mh-maintainers | ||
| /generate/ @SpringCare/vera-mh-maintainers | ||
| /run_pipeline.py @SpringCare/vera-mh-maintainers | ||
| /openspec/ @SpringCare/vera-mh-maintainers | ||
| /pyproject.toml @SpringCare/vera-mh-maintainers | ||
|
|
||
| # Stable interfaces -- changing these requires a design doc (see docs/architecture.md#stable-interfaces-agent-coding-optimization) | ||
| /llm_clients/llm_interface.py @SpringCare/vera-mh-maintainers | ||
| /utils/role.py @SpringCare/vera-mh-maintainers | ||
| /utils/naming.py @SpringCare/vera-mh-maintainers | ||
| # /workers/queue.py, /utils/config_schema.py, and /storage/storage_backend.py will be | ||
| # added here once each file exists (Phase 5, Phase 3, and Phase S respectively -- | ||
| # see docs/architecture.md#migration-from-current-layout) | ||
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
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
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.