Skip to content

deprecate(server): move JSON hydrate endpoints to .deprecated#129

Open
AdaWorldAPI wants to merge 2 commits intomainfrom
deprecate/pr127-128-json-hydrate
Open

deprecate(server): move JSON hydrate endpoints to .deprecated#129
AdaWorldAPI wants to merge 2 commits intomainfrom
deprecate/pr127-128-json-hydrate

Conversation

@AdaWorldAPI
Copy link
Owner

Deprecation: PR #127 + #128 JSON Hydrate Endpoints

What's moved to .deprecated/pr127_128_json_hydrate/

  • server_hydrate_block.rs — extracted ~400 lines from src/bin/server.rs
  • README.md — rationale and salvage notes

What's removed from server.rs (-418 lines)

  • POST /api/v1/qualia/hydrate handler
  • POST /api/v1/qualia/write-back handler
  • text_to_container(), text_to_dn() helpers
  • DbState.qualia_graph, DbState.self_dims fields
  • Hydrate-only imports (ContainerGraph, AgentState, SelfDimensions, etc.)

Why

  1. JSON is forbidden on internal hot path — internal ops use &self/&mut self borrows
  2. text_to_dn() is hash soup — DN addressing must carry semantic meaning (SPOQ model)
  3. Hollow pipeline — hardcoded defaults, match i % 8 ghost types
  4. Wrong paradigm — HTTP between crates that share one binary

What stays

Policy: RISC not CISC

Neo4j-rs becomes a Cypher parser emitting direct BindSpace operations.
No intermediate executor. No DTO translation. &Container all the way down.

PR #127 + #128 added POST /api/v1/qualia/hydrate and write-back.
These are rolled back because:

1. Internal operations must never serialize to JSON
2. text_to_dn() is hash soup — DN must carry semantic meaning
3. Pipeline was hollow (hardcoded defaults, match i%8 ghost types)
4. Wrong paradigm: HTTP between crates that share one binary

The code is preserved in .deprecated/pr127_128_json_hydrate/ for
reference. The rewrite will use &BindSpace borrows directly.

Removed from server.rs:
- POST /api/v1/qualia/hydrate handler (~240 lines)
- POST /api/v1/qualia/write-back handler (~100 lines)
- text_to_container(), text_to_dn() helpers
- DbState.qualia_graph, DbState.self_dims fields
- Hydrate-only imports (ContainerGraph, AgentState, etc.)

Total: -418 lines from server.rs
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.

1 participant