Skip to content

Clean Thingifier DDD and HTTP boundaries#37

Merged
eviltester merged 22 commits into
masterfrom
codex/replace-simplequery-repository-resolution
Jul 13, 2026
Merged

Clean Thingifier DDD and HTTP boundaries#37
eviltester merged 22 commits into
masterfrom
codex/replace-simplequery-repository-resolution

Conversation

@eviltester

Copy link
Copy Markdown
Owner

Summary

  • split repository/store responsibilities and keep relationship cascade decisions inside repository/store boundaries
  • add application command services and shared relationship parsing/resolution
  • clean Spark to internal HTTP conversion with explicit internal request/response types

Verification

  • mvn -pl thingifier,challenger -am verify
  • mvn -pl challengerAuto -am test "-Dchallenger.auto.local.repository=memory" "-DfailIfNoTests=false"
  • mvn -pl challengerAuto -am test "-Dchallenger.auto.local.repository=sqlite-memory" "-DfailIfNoTests=false"

Copilot AI review requested due to automatic review settings July 13, 2026 15:57
@eviltester eviltester merged commit 5f692c1 into master Jul 13, 2026
0 of 2 checks passed

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 wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

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.

2 participants