Skip to content

improve logging and simplify deployment routing behavior#34

Merged
jimmieme merged 1 commit intoao-space:devfrom
jimmieme:dev
Feb 6, 2026
Merged

improve logging and simplify deployment routing behavior#34
jimmieme merged 1 commit intoao-space:devfrom
jimmieme:dev

Conversation

@jimmieme
Copy link
Member

@jimmieme jimmieme commented Feb 6, 2026

Summary

This PR improves space-agent observability and stability for simplified
deployments, especially LAN-first and compose-driven scenarios.

What Changed

  • Added structured HTTP access logs (method, URI, status, latency, client IP,
    request ID).
  • Replaced scattered fmt output with consistent logger-based messages.
  • Added EnableDockerManage to allow running agent APIs without forcing docker
    service orchestration.
  • Added URL normalization for internal service calls in single-docker mode
    (localhost -> compose service name).
  • Improved /agent/info host persistence with host validation (skip localhost/
    loopback/empty hosts).
  • Added fallback logic for BTID/SN/QR generation when hardware values are
    missing.
  • Added router contract tests to ensure key internal/external APIs remain
    registered.

Why

  • Simplified deployment mode needs predictable routing and better logs for
    troubleshooting.
  • Existing behavior could bind to invalid loopback addresses in containerized
    scenarios.
  • Missing hardware identifiers should degrade gracefully instead of breaking
    bind/QR flows.

Validation

  • Verified agent starts with improved startup/access logs.
  • Verified router contracts via routes_contract_test.go.
  • Verified LAN host handling and fallback identity generation paths in
    simplified deployment flows.

Generated-by: Codex

@jimmieme jimmieme merged commit 5296256 into ao-space:dev Feb 6, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant