Skip to content

Expose shared document library state#177

Merged
suguanYang merged 4 commits into
mainfrom
feat/suguan/shared-document-library
Jun 26, 2026
Merged

Expose shared document library state#177
suguanYang merged 4 commits into
mainfrom
feat/suguan/shared-document-library

Conversation

@suguanYang

@suguanYang suguanYang commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds document_metadata to canonical documents and job creation metadata, then copies it through publication and exposes it in document responses.
  • Keeps GET /api/v1/documents document-only; parsing state remains on job creation/read responses rather than being projected into document listing.
  • Returns the planned document_id from job creation and job read responses before publication, so clients can correlate active parsing state immediately.
  • Removes the namespace migration/backfill helper in favor of client-side compatibility between default and legacy Notebook namespaces.

Part of #176.

Testing

  • uv run --all-packages pytest apps/api/tests/contract/test_documents_contract.py apps/api/tests/contract/test_job_creation_contract.py apps/api/tests/contract/test_job_read_contract.py -q
  • uv run pytest apps/api/tests apps/worker/tests/contract -q
  • make check
  • git diff --check

Comment thread apps/api/tests/contract/test_document_namespace_migration.py Fixed
@suguanYang suguanYang merged commit 1640056 into main Jun 26, 2026
6 checks passed
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