Skip to content

itdove/lore#83: Refactor: replace level magic numbers with LevelPolicy object - #86

Merged
itdove merged 1 commit into
mainfrom
83
Jul 27, 2026
Merged

itdove merged 1 commit into
mainfrom
83

Conversation

@itdove

@itdove itdove commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Jira Issue: #83

Description

#83: Refactor: replace level magic numbers with LevelPolicy object

Assisted-by: Claude

Testing

Steps to test

  1. Pull down the PR and verify changes build successfully
  2. Review the changed files for correctness
  3. Run the project's test suite

Scenarios tested

Deployment considerations

  • This code change is ready for deployment on its own
  • This code change requires the following considerations before being deployed:

- Add INDIVIDUAL_LEVEL, PROJECT_LEVEL, IMPLICIT_LEVELS constants and
  LevelPolicy dataclass to config/models.py
- Replace hardcoded level checks (< 2) with IMPLICIT_LEVELS membership
- Extract _submit_pr helper and refactor _resolve_level to return
  LevelPolicy with full write/PR/delete semantics
- Update MCP tools, sync engine, and CLI to use LevelPolicy for
  level-aware storage and PR decisions

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

Co-Authored-By: Claude <noreply@anthropic.com>
@itdove
itdove marked this pull request as ready for review July 27, 2026 13:35
@itdove
itdove merged commit dd70b9e into main Jul 27, 2026
6 checks passed
@itdove
itdove deleted the 83 branch July 27, 2026 13:35
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