Skip to content

fix(worker): resolve startup crash#30

Merged
sahiee-dev merged 2 commits intomainfrom
fix/worker-stability
Mar 7, 2026
Merged

fix(worker): resolve startup crash#30
sahiee-dev merged 2 commits intomainfrom
fix/worker-stability

Conversation

@sahiee-dev
Copy link
Copy Markdown
Owner

@sahiee-dev sahiee-dev commented Feb 18, 2026

resolve startup crash

  • Use .policy_hash instead of .source_hash in logging statement
  • Fixes AttributeError during worker init

Summary by CodeRabbit

  • Chores
    • Updated logging to display policy hash instead of source hash when initializing the policy engine.

- Use .policy_hash instead of .source_hash in logging statement
- Fixes AttributeError during worker init
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 18, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7ec4996c-1a29-4051-9cc0-55169182b09e

📥 Commits

Reviewing files that changed from the base of the PR and between b4b0ee3 and 996e0d4.

📒 Files selected for processing (1)
  • backend/app/worker/main.py

📝 Walkthrough

Walkthrough

A logging modification in the PolicyEngine initialization that changes the reported identifier from source hash to policy hash for each initialized policy. No functional or behavioral changes to the system.

Changes

Cohort / File(s) Summary
Policy Logging Update
backend/app/worker/main.py
Modified policy initialization logging to report policy hash instead of source hash when listing policies in the PolicyEngine.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hop and a change so small and neat,
The policy hash now makes logging complete!
No bugs to squash, just clarity's gain,
One simple swap in the worker's domain.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/worker-stability

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sahiee-dev sahiee-dev self-assigned this Feb 18, 2026
@sahiee-dev sahiee-dev added the fixes/Verification used for additional fixes or verification ... label Feb 18, 2026
@sahiee-dev sahiee-dev merged commit 22a4387 into main Mar 7, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fixes/Verification used for additional fixes or verification ...

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant