Skip to content

🐛 fix: set api key in initialization#415

Merged
Nicola Franco (franconicola) merged 1 commit into
mainfrom
413-initialization-lacks-api-key-configuration
Jun 5, 2026
Merged

🐛 fix: set api key in initialization#415
Nicola Franco (franconicola) merged 1 commit into
mainfrom
413-initialization-lacks-api-key-configuration

Conversation

@marcorusso97

Copy link
Copy Markdown
Contributor

Summary

This PR improves the tool configuration.

  1. The hackagent init flow now supports remote-mode setup directly.
  2. The hackagent web command now opens the cloud app in remote mode instead of launching a local dashboard server.
  3. Installation documentation has been updated to reflect both behaviors.

What Changed

CLI Init Improvements

  • Added an explicit mode selection step in hackagent init:
    • Local mode (default)
    • Remote mode
  • The remote mode prompt now defaults to No (y/N).
  • In remote mode, the wizard asks only for the API key.
  • If remote mode is selected but no API key is provided, setup falls back to local mode.
  • Setup completion messaging now reflects whether local or remote mode is active.

Web Command Remote Behavior

  • hackagent web now detects remote mode (API key configured) and opens:
    • https://app.hackagent.dev
  • In remote mode, it no longer starts the local Flask/NiceGUI dashboard server.
  • Local mode behavior is unchanged: it still starts the local dashboard server.
  • --no-browser is respected in remote mode (prints URL without auto-opening).

Documentation Updates

  • Updated Quick Install to describe:
    • mode selection in hackagent init
    • API-key-based remote setup
    • hackagent web behavior in local vs remote mode

Tests

Added/updated unit coverage for:

  • init wizard remote/local flows
  • web command behavior in:
    • remote mode (opens cloud URL)
    • remote mode + --no-browser
    • local mode (launches local dashboard)

@marcorusso97 Marco Russo (marcorusso97) linked an issue Jun 3, 2026 that may be closed by this pull request
@franconicola Nicola Franco (franconicola) temporarily deployed to 413-initialization-lacks-api-key-configuration - Docs PR #415 June 3, 2026 08:46 — with Render Destroyed
@codecov

codecov Bot commented Jun 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.09677% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
hackagent/cli/main.py 81.25% 3 Missing ⚠️
hackagent/cli/commands/web.py 93.33% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@franconicola Nicola Franco (franconicola) merged commit 1613cb5 into main Jun 5, 2026
24 checks passed
@franconicola Nicola Franco (franconicola) deleted the 413-initialization-lacks-api-key-configuration branch June 5, 2026 13:17
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.

Initialization lacks api key configuration

2 participants