Add automated WebArena map backend deployment infrastructure

[neubig]

Overview

This PR makes several improvements to make webarena deployment easier:

1. Provides instructions about how to deploy the map backend server on AWS with the data stored in an s3 bucket
2. Uses a web server deployed on AWS instead of metis/ogma for the default map backend server
3. Fixes CI by removing CI tests that assume that a particular web site is live and fixing mypy errors
4. Updates that AMI so that the configuration process is more straightforward

[openhands-ai]

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

• GitHub Actions are failing:
  • Python Package Pytest

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

@OpenHands please fix the failing actions on PR #217 at branch `feat/automated-map-backend-deployment`

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

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

[Copilot]

Pull Request Overview

This PR enhances WebArena deployment infrastructure with automated AWS-based map backend deployment and improves CI reliability. It provides a complete cloud-init script for setting up tile servers, geocoding, and routing services using pre-configured data from S3, while replacing hardcoded URLs with environment variable-based configuration.

• Adds automated map backend deployment infrastructure with cloud-init script
• Removes CI tests that depend on external website availability
• Improves configuration management with environment variable validation

Reviewed Changes

Copilot reviewed 23 out of 25 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
webarena-map-backend-boot-init.yaml	Complete cloud-init script for automated map backend deployment
tests/test_evaluation_harness/	Removes tests dependent on external websites
setup_env.sh	New environment variable configuration script
minimal_example.py	Replaces hardcoded URLs with environment variable validation
setup.cfg	Adds mypy configuration and missing type stubs
browser_env/actions.py	Fixes mypy type annotation issues
.github/workflows/tests.yml	Removes hardcoded environment variables from CI

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

The script uses the AWS IMDS endpoint without specifying a token, which makes it vulnerable to SSRF attacks. Consider using IMDSv2 with session tokens for improved security.

Suggested change

	echo " - Tile server: http://$(curl -s http://169.254.169.254/latest/meta-data/public-ipv4):8080/tile/{z}/{x}/{y}.png"
	echo " - Geocoding: http://$(curl -s http://169.254.169.254/latest/meta-data/public-ipv4):8085/"
	echo " - OSRM Car: http://$(curl -s http://169.254.169.254/latest/meta-data/public-ipv4):5000/"
	echo " - OSRM Bike: http://$(curl -s http://169.254.169.254/latest/meta-data/public-ipv4):5001/"
	echo " - OSRM Foot: http://$(curl -s http://169.254.169.254/latest/meta-data/public-ipv4):5002/"
	# Use IMDSv2 for secure metadata access
	IMDS_TOKEN=$(curl -X PUT "http://169.254.169.254/latest/api/token" -H "X-aws-ec2-metadata-token-ttl-seconds: 60" -s)
	INSTANCE_IP=$(curl -s -H "X-aws-ec2-metadata-token: $IMDS_TOKEN" http://169.254.169.254/latest/meta-data/public-ipv4)
	echo " - Tile server: http://$INSTANCE_IP:8080/tile/{z}/{x}/{y}.png"
	echo " - Geocoding: http://$INSTANCE_IP:8085/"
	echo " - OSRM Car: http://$INSTANCE_IP:5000/"
	echo " - OSRM Bike: http://$INSTANCE_IP:5001/"
	echo " - OSRM Foot: http://$INSTANCE_IP:5002/"

[Copilot]

This complex port calculation logic is difficult to understand and maintain. Consider using a simple associative array or case statement for better readability.

Suggested change

	for service in car bike foot; do
	port=$((5000 + $(echo "car bike foot" | tr ' ' '\n' | grep -n $service | cut -d: -f1) - 1))
	# Define service ports using associative array for clarity
	declare -A service_ports=( ["car"]=5000 ["bike"]=5001 ["foot"]=5002 )
	for service in car bike foot; do
	port=${service_ports[$service]}

[Copilot]

Converting non-string AST nodes to strings using str(arg) will produce AST object representations instead of actual values. This should handle different AST node types appropriately or use ast.unparse() for proper code generation.

Suggested change

	str(ast.literal_eval(arg))
	if isinstance(arg, ast.Str)
	else str(arg)
	ast.unparse(arg)