treat mpt as default uri scheme in cogames

[nishu-builder]

With this change, you can specify cogames -p with URIs that resolve to MPTs:

uv run cogames play -m hello_world -p s3://softmax-public/policies/daveey.4x4.cvc.dr.maps.1.bct.0_9/:latest

uv run cogames play -m hello_world -p metta://policy/daveey.4x4.cvc.dr_v1_rem.c_8

[nishu-builder]

• treat mpt as default uri scheme in cogames #4089 👈 (View in Graphite)
• Refactor a bunch of stuff around uri schemes #4088
• main

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

[chatgpt-codex-connector]

💡 Codex Review

metta/packages/cogames/src/cogames/cli/policy.py

Lines 136 to 140 in ec22952

	try:
	resolved = resolve_uri(raw)
	if resolved.endswith(".mpt"):
	base_spec = policy_spec_from_uri(raw)
	return PolicySpecWithProportion(

Handle non-ValueError failures from URI resolution

The new URI path in _parse_policy_spec calls resolve_uri/policy_spec_from_uri, which can raise resolver exceptions that are not ValueError (e.g., botocore.exceptions.NoCredentialsError when -p s3://... is used without AWS credentials). The surrounding try/except only handles ValueError, so those exceptions now bubble out past get_policy_spec’s handler and crash the CLI instead of showing the friendly parse error/help output. Please catch and translate resolver failures here so policy URIs fail gracefully.

ℹ️ 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".