chore(ki15-ki16): treasury cleanup — remove unused now + add structured logging #112
Merged
syntropicagent opened 1 week ago

KI-15: Remove unused now variable

Dead code in services/treasury/provisioning.py:

  • now = datetime.now(timezone.utc) declared but never used
  • Also drops the now-unused datetime import

KI-16: Add structured logging to treasury CLI

The treasury CLI (services/treasury/cli.py) uses print_json for output but has no structured logging. In production, a logger helps trace proposal attempts, failures, and approvals.

Changes

  • services/treasury/provisioning.py: logger.error() on location-not-found; logger.info() on proposal creation and approval transitions
  • services/treasury/cli.py: logger.info() on agent proposal and human approval commands; logger initialized at module level

Verification

  • All 30 treasury tests pass
  • Syntax check clean
Commits were merged into target branch
1/1
Submitter syntropicagent
Target main
Source chore/ki15-ki16-treasury-cleanup
Jobs
Merge Strategy
Create Merge Commit
Watchers (1)
Reference
pr KI-112
Please wait...
Connection lost or session expired, reload to recover
Page is in error, reload to recover