Skip to content

Commit aae1e85

Browse files
author
LittleCoinCoin
committed
fix(cli): string value usage
Several places in `cli_hatch` were expecting `host` to be an enum and therefore called `host.value` when in need of the string value. But this is already a string.
1 parent 5638299 commit aae1e85

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

hatch/cli_hatch.py

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1387,7 +1387,7 @@ def main():
13871387

13881388
# Configure on each host
13891389
success_count = 0
1390-
for host in hosts:
1390+
for host in hosts: # 'host', here, is a string
13911391
try:
13921392
result = mcp_manager.configure_server(
13931393
hostname=host,
@@ -1396,13 +1396,13 @@ def main():
13961396
)
13971397

13981398
if result.success:
1399-
print(f"✓ Configured {server_config.name} on {host.value}")
1399+
print(f"✓ Configured {server_config.name} on {host}")
14001400
success_count += 1
14011401
else:
1402-
print(f"✗ Failed to configure {server_config.name} on {host.value}: {result.error_message}")
1402+
print(f"✗ Failed to configure {server_config.name} on {host}: {result.error_message}")
14031403

14041404
except Exception as e:
1405-
print(f"✗ Error configuring {server_config.name} on {host.value}: {e}")
1405+
print(f"✗ Error configuring {server_config.name} on {host}: {e}")
14061406

14071407
if success_count > 0:
14081408
print(f"MCP configuration completed: {success_count}/{len(hosts)} hosts configured")
@@ -1448,7 +1448,7 @@ def main():
14481448
server_config = get_package_mcp_server_config(env_manager, env_name, args.package_name)
14491449

14501450
if args.dry_run:
1451-
print(f"[DRY RUN] Would synchronize MCP server for package '{args.package_name}' to hosts: {[h.value for h in hosts]}")
1451+
print(f"[DRY RUN] Would synchronize MCP server for package '{args.package_name}' to hosts: {[h for h in hosts]}")
14521452
print(f"[DRY RUN] Server config: {server_config.name} -> {' '.join(server_config.args)}")
14531453
return 0
14541454

@@ -1462,16 +1462,16 @@ def main():
14621462

14631463
# Perform synchronization to each host
14641464
success_count = 0
1465-
for host in hosts:
1465+
for host in hosts: # 'host', here, is a string
14661466
try:
14671467
result = mcp_manager.configure_server(
1468-
hostname=host.value, # Use enum value (string) instead of enum object
1468+
hostname=host,
14691469
server_config=server_config,
14701470
no_backup=args.no_backup
14711471
)
14721472

14731473
if result.success:
1474-
print(f"[SUCCESS] Successfully configured {server_config.name} on {host.value}")
1474+
print(f"[SUCCESS] Successfully configured {server_config.name} on {host}")
14751475
success_count += 1
14761476

14771477
# Update package metadata with host configuration tracking
@@ -1485,17 +1485,17 @@ def main():
14851485
env_manager.update_package_host_configuration(
14861486
env_name=env_name,
14871487
package_name=args.package_name,
1488-
hostname=host.value,
1488+
hostname=host,
14891489
server_config=server_config_dict
14901490
)
14911491
except Exception as e:
14921492
# Log but don't fail the sync operation
14931493
print(f"[WARNING] Failed to update package metadata: {e}")
14941494
else:
1495-
print(f"[ERROR] Failed to configure {server_config.name} on {host.value}: {result.error_message}")
1495+
print(f"[ERROR] Failed to configure {server_config.name} on {host}: {result.error_message}")
14961496

14971497
except Exception as e:
1498-
print(f"[ERROR] Error configuring {server_config.name} on {host.value}: {e}")
1498+
print(f"[ERROR] Error configuring {server_config.name} on {host}: {e}")
14991499

15001500
# Report results
15011501
if success_count == len(hosts):

0 commit comments

Comments
 (0)