Alan-Jowett / CoPilot-For-Consensus / 20326935572

78%

push

github

Complete standardization of canonical identifiers across codebase (issue #372) (#381)

* docs: add identifier standardization analysis (issue #372)

Created comprehensive inventory and analysis document covering:
- Current state of _id/_key usage across services and databases
- Identified issues with UUID-based summary_id and report_id
- Key derivation rules (archive_id, message_key, chunk_key, thread_id)
- 5-phase migration plan with specific files and tasks
- Acceptance criteria and next steps

* docs: adopt _id as canonical deterministic primary key (issue #372)\n\n- Make _id the single source of truth across collections\n- Update derivation rules, migration plan, acceptance criteria\n- Align services, schemas, and tests with _id-centric approach

* docs/schemas: make _id canonical\n\n- SCHEMA.md: switch archives/messages/chunks/threads to  primary key\n- archives.schema.json: add  (required), deprecate

* schemas: messages use  as canonical; deprecate \n\n- Add required \n- Keep  as deprecated alias for transition

* schemas(messages): remove message_key (breaking change)\n\n- Drop message_key from messages schema\n-  is the sole primary identifier

* adapter(schema_validation): remove legacy message_key generator; migrate tests to canonical _id; update event docstrings; align schemas/docs for #372

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* events(schema): align event schemas to canonical identifier constraints (16-64 hex) and descriptions; no field renames

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* events(schema): rename message_keys -> message_doc_ids (breaking); update model docs; enforce canonical hex patterns

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* Migrate chunking service to canonical identifiers

- Replace message_key → message_doc_id (_id) throughout service and tests
- Replace message_keys → message_doc_ids in event payloads
- Replace chunk_key/chunk_id → _id for chunk document primary ke... (continued)

111 of 126 new or added lines in 11 files covered. (88.1%)

4453 of 5377 relevant lines covered (82.82%)

0.93 hits per line