Rename cog_arena.py to cogsguard.py and remove unused imports

[daveey]

TL;DR

Renamed cog_arena.py to cogsguard.py and removed unused imports.

What changed?

• Renamed the file from recipes/experiment/cog_arena.py to recipes/experiment/cogsguard.py
• Removed several unused imports:
  • AOEEffectConfig
  • BuildConfig
  • DemolishConfig
  • MarketConfig
  • MarketTerminalConfig
  • TransferActionConfig
  • VibeTransfer

How to test?

1. Verify that any imports from the old file path are updated to reference the new file path
2. Ensure that the code still compiles and runs correctly without the removed imports
3. Check that no functionality is broken by the file rename

Why make this change?

This change improves code cleanliness by removing unused imports and renames the file to better reflect its purpose or align with a new naming convention. The rename from cog_arena to cogsguard suggests a more specific focus or branding for this component.

[daveey]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• cp #4566
• Add align action and update related action handlers #4565
• Add observer_agent_id parameter to obs_features for agent-specific observations #4539
• Add Area of Effect (AOE) resource effects to grid objects #4538
• Add CommonsChest for shared inventory access across agents #4537
• Add Commons system for shared inventories between grid objects #4536
• Rename cog_arena.py to cogsguard.py and remove unused imports #4477 👈 (View in Graphite)
• Enhance attack system with armor, weapons, and configurable loot #4313 : 1 other dependent PR (#4321 )
• Make Transfer action optional and add vibe-triggered attack #4315
• Implement bidirectional resource transfer system with vibe-based triggers #4314
• feat: support metta://policy/<short_name_or_class_path> URIs for built-in policies #4561
• feat: allow negative inventory_regen_amounts for decay #4564
• Add clarification about when to run install commands #4563
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[chatgpt-codex-connector]

💡 Codex Review

metta/recipes/experiment/cogsguard.py

Lines 148 to 152 in a60bb7e

	transfer=TransferActionConfig(
	vibe_transfers=[
	VibeTransfer(vibe="battery", target={"energy": 50}, actor={"energy": -50}),
	VibeTransfer(vibe="heart", target={"heart": 1}, actor={"heart": -1}),
	VibeTransfer(vibe="gear", target={"damage": -100}, actor={"energy": -10}),

Reintroduce transfer action imports

TransferActionConfig and VibeTransfer are still used to configure the transfer action in make_env, but they were dropped from the mettagrid imports above. With the imports removed, calling make_env now raises NameError: TransferActionConfig is not defined before the environment can be built. Restore the imports or remove the transfer block to keep the recipe loadable.

---

metta/recipes/experiment/cogsguard.py

Lines 215 to 219 in a60bb7e

	"market": MarketConfig(
	terminals={
	# Terminals are keyed by the direction the agent enters the market from.
	"north": MarketTerminalConfig(sell=False, amount=1), # buy
	"south": MarketTerminalConfig(sell=True, amount=1), # sell

Market config names now undefined at runtime

The market object still instantiates MarketConfig and MarketTerminalConfig, but those imports were removed from the mettagrid import list. When make_env reaches this block it will fail immediately with NameError rather than producing a valid environment. Add the missing imports or remove these usages so the recipe can be constructed.

---

metta/recipes/experiment/cogsguard.py

Line 238 in a60bb7e

build=BuildConfig(vibe="barrier", cost={"energy": 50, "silicon": 25}),

Barrier build config references undefined class

BuildConfig (and the related demolition/aoe configs) are still referenced in the barrier definition, but the corresponding imports were removed. The first access at build=BuildConfig(...) will raise NameError, preventing make_env from returning. Reintroduce the missing mettagrid imports to keep the barrier object usable.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[openhands-ai]

Looks like there are a few issues preventing this PR from being merged!

• GitHub Actions are failing:
  • Test and Benchmark

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #4477 at branch `daveey-cogsguard`

Feel free to include any additional details that might help me get this PR into a better state.

_{^{You can manage your notification settings}}