Skip to content

Remove persisted reference simulation semantics#216

Merged
tomvothecoder merged 2 commits into
mainfrom
refactor/195-remove-ref-model
Jun 4, 2026
Merged

Remove persisted reference simulation semantics#216
tomvothecoder merged 2 commits into
mainfrom
refactor/195-remove-ref-model

Conversation

@tomvothecoder

Copy link
Copy Markdown
Collaborator

Description

  • Removes persisted reference simulation state and run-config delta payloads from the backend and frontend.
  • Keeps case_hash as per-run grouping metadata and simplifies case/browse/details flows around neutral compare behavior.
  • Updates the migration, seed data, docs, and ingestion/assistant tests to match the new model.
  • Closes [Enhancement]: Remove reference model and use case-hash-scoped compare #195

Checklist

  • Code follows project style guidelines
  • Self-reviewed code
  • No new warnings
  • Tests added or updated (if needed)
  • All tests pass (locally and CI/CD)
  • Documentation/comments updated (if needed)
  • Breaking change noted (if applicable)

Deployment Notes (if any)

  • Run the migration 20260604_120000_remove_reference_and_delta_fields with the backend deploy.
  • Backend and frontend should deploy together because simulation API fields were removed.

@tomvothecoder tomvothecoder changed the title [codex] remove persisted reference simulation semantics Remove persisted reference simulation semantics Jun 4, 2026
@tomvothecoder tomvothecoder requested a review from Copilot June 4, 2026 18:22

This comment was marked as outdated.

@tomvothecoder tomvothecoder marked this pull request as ready for review June 4, 2026 18:26
@tomvothecoder tomvothecoder requested a review from Copilot June 4, 2026 18:26

This comment was marked as outdated.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@tomvothecoder

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 👍

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@tomvothecoder tomvothecoder merged commit 34522d7 into main Jun 4, 2026
1 of 2 checks passed
@tomvothecoder tomvothecoder deleted the refactor/195-remove-ref-model branch June 4, 2026 18:55
Copilot stopped reviewing on behalf of tomvothecoder due to an error June 4, 2026 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enhancement]: Remove reference model and use case-hash-scoped compare

2 participants