Skip to content

feat: add BIRD individual lookup with prefix auto-detection - #11

Merged
N283T merged 4 commits into
mainfrom
feature/bird-individual-lookup
Mar 7, 2026
Merged

feat: add BIRD individual lookup with prefix auto-detection#11
N283T merged 4 commits into
mainfrom
feature/bird-individual-lookup

Conversation

@N283T

@N283T N283T commented Mar 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • Auto-detect PRD_/PRDCC_/FAM_ prefixes to route individual BIRD file lookups to correct subdirectory (prd/, prdcc/, family/)
  • Split bulk PRD into separate view-cif prd (prd-all) and view-cif prdcc (prdcc-all) commands
  • Remove --ccd-definition flag (no longer needed with prefix auto-detection)

Test plan

  • Unit tests for _bird_subdir prefix detection (including PRDCC vs PRD ordering)
  • Unit tests for individual PRD/PRDCC/FAM resolution
  • Unit tests for bulk PRD/PRDCC separation
  • Case-insensitive input handling
  • Missing path config error handling
  • All 51 tests passing, 81% coverage
  • Local integration test with real PDB mirror data

N283T added 4 commits March 7, 2026 21:03
- Auto-detect PRD_/PRDCC_/FAM_ prefixes to route to correct
  bird subdirectory (prd/, prdcc/, family/)
- Split bulk PRD into separate `prd` and `prdcc` commands
- Remove --ccd-definition flag (no longer needed)
- Add tests for bird prefix detection and individual lookups
- Use tuple for _BIRD_PREFIXES with explicit ordering (no sorted())
- Add None guard in _resolve_bird_individual for safety
- Update CLI help text to mention BIRD identifiers
- Add missing test for prdcc bulk missing path
Provide clear error message when bird subdirectory (prd/prdcc/family)
does not exist under paths.prd, instead of misleading "file not found".
@N283T
N283T merged commit cdc23b1 into main Mar 7, 2026
6 checks passed
@N283T
N283T deleted the feature/bird-individual-lookup branch March 7, 2026 12:11
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