Add configurable metadata field indexing with sync modes #53
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Implements environment-driven metadata field indexing configuration with four synchronization modes for handling index mismatches at server startup. This feature allows users to customize which metadata fields are indexed via METADATA_INDEXED_FIELDS environment variable with optional type hints (string, integer, boolean, float, array, object). The METADATA_INDEX_SYNC_MODE variable controls how index mismatches are handled: strict (fail), auto (sync), warn (log), or additive (default, add missing only). Key changes: - New METADATA_INDEXED_FIELDS env var for configuring indexed fields with type hints - New METADATA_INDEX_SYNC_MODE env var with four modes: strict, auto, warn, additive - Backend-aware index management supporting both SQLite and PostgreSQL - Updated default indexed fields: status, agent_name, task_name, project, report_type, references, technologies - GIN index support for array/object fields in PostgreSQL (skipped in SQLite) - Schema updates reflecting new indexed field defaults
Adds 38 tests covering all aspects of the configurable metadata field indexing feature including settings parsing, SQL generation, index detection, index creation/deletion, and all four sync modes. Test coverage includes: - Settings field parsing with type hints validation - SQLite and PostgreSQL SQL generation for different field types - Existing index detection excluding composite and GIN indexes - Index creation with backend-specific handling - All four sync modes: strict, auto, warn, additive - Integration tests for server startup behavior - Updated database test expectations for new default indexes
Updates all documentation to reflect the new configurable metadata field indexing feature including environment variables, backend differences, and indexed field defaults. Documentation updates include: - CLAUDE.md: Added metadata indexing settings and backend comparison table - README.md: Added metadata indexing configuration section - docs/metadata-addition-updating-and-filtering.md: Comprehensive guide with new indexed fields, environment variables, and backend differences - docs/api-reference.md: Updated performance notes with configurable indexing - docs/deployment/docker.md: Added metadata indexing environment variables table
Coverage report
Click to see where and how coverage changed
This report was generated by python-coverage-comment-action |
||||||||||||||||||||||||||||||
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.