Skip to content

fix: resolve calibration CodeQL import cycles - #260

Merged
suguanYang merged 3 commits into
mainfrom
fix/wangbinqi/resolve-pr259-review
Aug 12, 2026
Merged

fix: resolve calibration CodeQL import cycles#260
suguanYang merged 3 commits into
mainfrom
fix/wangbinqi/resolve-pr259-review

Conversation

@suguanYang

Copy link
Copy Markdown
Contributor

Resolves the CodeQL cyclic-import review comments from PR #259.

Changes:

  • move low-level anchoring implementation to anchoring_primitives.py
  • add calibration-owned orchestration in calibration/orchestrator.py
  • keep structure_anchoring compatibility exports and patch seams
  • update production callers and calibration imports to use the acyclic dependency direction

Verification:

  • python -m compileall passed
  • git diff --check passed
  • import smoke test passed with required settings
  • contract suite was attempted but the checkout is missing the declared fakeredis dependency

@suguanYang
suguanYang merged commit 076636e into main Aug 12, 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