fix: add missing import for importance_scheduler

[Cipher208]

The importance scheduler was being started in the lifespan but wasn't imported, causing a NameError on startup.

Summary by CodeRabbit

• Bug Fixes
  • Enabled the scheduler to start and stop correctly during server lifecycle events.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: f2800506-acb5-455b-b701-52fa87069555

📥 Commits

Reviewing files that changed from the base of the PR and between 705e000 and f56f483.

📒 Files selected for processing (1)

• mcp_server/server.py

---

Walkthrough

The server now imports importance_scheduler, enabling its existing lifecycle start and stop calls.

Changes

Scheduler lifecycle wiring

Layer / File(s)	Summary
Import scheduler for lifecycle calls mcp_server/server.py	The server imports importance_scheduler for background-task startup and shutdown.

Estimated code review effort: 1 (Trivial) | ~2 minutes

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/importance-scheduler-import

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[greptile-apps]

Greptile Summary

The PR imports the existing importance_scheduler singleton so the server’s startup and shutdown lifecycle references resolve correctly.

• Adds the missing direct import in mcp_server/server.py.
• Restores access to the scheduler’s existing start() and stop() interface.

Confidence Score: 5/5

The PR appears safe to merge, with no actionable defects identified in the added import.

The imported module exists, exports the expected singleton, has no incompatible circular dependency, and supplies the lifecycle methods already called by the server.

Important Files Changed

Filename	Overview
mcp_server/server.py	Adds the missing scheduler import; the target module and exported singleton exist, and no circular dependency or import-time incompatibility was identified.

_{Reviews (1): Last reviewed commit: "fix: add missing import for importance_s..." | Re-trigger Greptile}