Skip to content

Conversation

@marcarl
Copy link
Collaborator

@marcarl marcarl commented Jan 7, 2026

Summary

  • Add support for handling multiple CELEX numbers (comma or space-separated)
  • Single CELEX numbers are stored as strings, multiple as YAML lists
  • Applied to both main documents and amendments

Changes

  • Add parse_celex_numbers() function to split CELEX strings by comma/space
  • Update YAML front matter generation to output single value or list format
  • Apply same parsing logic to amendment CELEX numbers in temporal/amendments.py

Test plan

  • Test with single CELEX number (should output as string)
  • Test with multiple CELEX numbers separated by comma
  • Test with multiple CELEX numbers separated by space
  • Verify YAML output format is valid

🤖 Generated with Claude Code

marcarl and others added 2 commits January 7, 2026 19:59
Add support for handling multiple CELEX numbers (comma or space-separated)
in both main documents and amendments. When multiple CELEX numbers are
present, they are now stored as a YAML list instead of a single string.

Changes:
- Add parse_celex_numbers() function to split CELEX strings
- Update YAML front matter to output single value or list
- Apply same parsing to amendment CELEX numbers

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
The frontmatter sorter now properly handles CELEX numbers in amendments:
- Add 'celex' to AMENDMENT_ORDER for proper field ordering
- Parse nested YAML lists (e.g., multiple CELEX numbers)
- Generate correct YAML output for both single values and lists

This completes the CELEX number support by ensuring that frontmatter
sorting preserves CELEX data in the correct format.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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