♻️ Removing msg! in approve_system to allow up to 126 systems …

[notdanilo]

Status	Type	⚠️ Core Change	Issue
Ready	Refactor	No	No

Problem

msg! consumes too much CU and we couldn't approve more than 16 systems because of it.

Solution

Remove it. Now we can approve 126 systems.

Summary by CodeRabbit

• Chores
  • Removed unnecessary debug logging during system approval and removal to reduce console noise.
  • No functional changes; existing workflows and outcomes remain unchanged.
  • Users may notice cleaner, more focused logs during these operations, improving readability and troubleshooting.
  • No configuration changes required.

[coderabbitai]

Walkthrough

Removed debugging/logging statements from approve_system and remove_system in crates/programs/world/src/lib.rs without altering functionality or public APIs.

Changes

Cohort / File(s)	Summary
Logging removal crates/programs/world/src/lib.rs	Removed log statements in approve_system and remove_system; no functional or API changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title explicitly describes the removal of the msg! call in approve_system to increase the limit to 126 systems, which matches the primary change in the PR and clearly conveys its purpose. It is concise and focused on the main refactor.
Description Check	✅ Passed	The description includes the required status/type table and clearly outlines both the problem and the solution, which are the critical sections from the template. While it omits non-critical sections like screenshots, other changes, and deploy notes, the core information is adequately documented.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/approve-system

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f5186d9 and bde3c09.

📒 Files selected for processing (1)

• crates/programs/world/src/lib.rs (0 hunks)

💤 Files with no reviewable changes (1)

• crates/programs/world/src/lib.rs

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: test
• GitHub Check: setup

---

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

[GabrielePicco]

LGTM!