Skip to content

fix: replace localhost help_url defaults with production URL#92

Merged
BeckettFrey merged 2 commits intoreleasefrom
copilot/update-appconfig-help-url-default
Apr 13, 2026
Merged

fix: replace localhost help_url defaults with production URL#92
BeckettFrey merged 2 commits intoreleasefrom
copilot/update-appconfig-help-url-default

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 13, 2026

AppConfig.help_url fell back to http://localhost:3000/help when the YAML key was absent or the dataclass was constructed without an explicit value — silently breaking the help button (webbrowser.open) for end users.

Changes

  • src/voxkit/config/app_config.py — corrected both the dataclass field default and the from_yaml fallback:
    # before
    help_url: str = "http://localhost:3000/help"
    help_url=data.get("help_url", "http://localhost:3000/help"),
    
    # after
    help_url: str = "https://voxkit-web.vercel.app/help"
    help_url=data.get("help_url", "https://voxkit-web.vercel.app/help"),
  • src/voxkit/config.py — updated the stale legacy HELP_URL constant to the production URL
  • tests/config/test_app_config.py — updated two default-value assertions to match the corrected URL

Testing

  • Test testable logic
  • Edge cases addressed

Testing Notes: Updated assertions in test_dataclass_defaults and test_from_yaml_with_defaults now validate the correct production fallback URL.

Documentation

  • README updated (if applicable)
  • Code comments added/updated
  • User-facing documentation updated (if applicable)

Documentation Notes: No documentation changes required; all YAML configs already use the correct production URL.

Checklist

  • Code follows project conventions
  • No breaking changes (or documented if intentional)
  • All CI checks pass

Copilot AI changed the title [WIP] Update AppConfig help_url default to production URL fix: replace localhost help_url defaults with production URL Apr 13, 2026
Copilot AI requested a review from BeckettFrey April 13, 2026 19:46
@BeckettFrey BeckettFrey marked this pull request as ready for review April 13, 2026 20:26
@BeckettFrey BeckettFrey merged commit c0024c0 into release Apr 13, 2026
3 checks passed
@BeckettFrey BeckettFrey deleted the copilot/update-appconfig-help-url-default branch April 13, 2026 20:27
BeckettFrey added a commit that referenced this pull request Apr 14, 2026
* fixes #64: add ai optimized documentation file

* fix: replace localhost help_url defaults with production URL (#92)

* Initial plan

* fix: replace localhost help_url defaults with production URL

Agent-Logs-Url: https://github.com/BrainBehaviorAnalyticsLab/voxkit-desktop/sessions/496f7ed1-fdaa-4df9-b574-5ab55254c136

Co-authored-by: BeckettFrey <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: BeckettFrey <[email protected]>

* fix: dataset panel empty state non-responsive when splitter resized (#94)

* Initial plan

* fix: prevent helper_label and empty_label from resizing with splitter

Agent-Logs-Url: https://github.com/BrainBehaviorAnalyticsLab/voxkit-desktop/sessions/ede8fa76-ccf4-4940-bd76-7f46adde44c5

Co-authored-by: BeckettFrey <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: BeckettFrey <[email protected]>

* setup-logging (#95)

* Refactor configuration and update release workflow process (#96)

* refactor: remove shadowed config

* replace: move to invoke for os agnostic clarity

* remove release workflow in favor of more manual steps

* Configure shredguard for blocking regex patterns (#98)

* fix: don't mark first launch complete on startup script error (#99)

* Initial plan

* fix: don't mark first launch complete on startup script error

Agent-Logs-Url: https://github.com/BrainBehaviorAnalyticsLab/voxkit-desktop/sessions/e51a7557-a81e-4972-a7e6-45133b115413

Co-authored-by: BeckettFrey <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: BeckettFrey <[email protected]>

* fix: readable_from_unique_id handles prefixed IDs from generate_unique_id (#101)

* Initial plan

* fix: handle prefixed IDs in readable_from_unique_id

Agent-Logs-Url: https://github.com/BrainBehaviorAnalyticsLab/voxkit-desktop/sessions/f6936450-f9a2-4df8-b75a-b403d98db565

Co-authored-by: BeckettFrey <[email protected]>

* fix: raise descriptive ValueError when no timestamp found in readable_from_unique_id

Agent-Logs-Url: https://github.com/BrainBehaviorAnalyticsLab/voxkit-desktop/sessions/f6936450-f9a2-4df8-b75a-b403d98db565

Co-authored-by: BeckettFrey <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: BeckettFrey <[email protected]>

* Initial plan (#103)

Co-authored-by: copilot-swe-agent[bot] <[email protected]>

* Revert "Initial plan (#103)" (#106)

This reverts commit b9ab4a0.

* Fix problem with dedicated internal function (#107)

* fix: add empty-ID guard to delete_model to prevent wiping engine models directory (#100)

* Initial plan

* fix: add empty-ID guard to delete_model to prevent wiping engine models directory

Agent-Logs-Url: https://github.com/BrainBehaviorAnalyticsLab/voxkit-desktop/sessions/0cce5a70-83a5-4c00-b1e8-19f85cb895d1

Co-authored-by: BeckettFrey <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: BeckettFrey <[email protected]>

* fix: validate_dataset checks stem-name pairing between audio and label files (#102)

* Initial plan

* fix: validate_dataset checks stem-name pairing between audio and label files

Agent-Logs-Url: https://github.com/BrainBehaviorAnalyticsLab/voxkit-desktop/sessions/73d34692-65d9-48a2-9621-7127982837a2

Co-authored-by: BeckettFrey <[email protected]>

* chore: remove accidentally committed root conftest.py

Agent-Logs-Url: https://github.com/BrainBehaviorAnalyticsLab/voxkit-desktop/sessions/46f1cfe0-1564-47a8-85ea-4c404b6b23f8

Co-authored-by: BeckettFrey <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: BeckettFrey <[email protected]>

* 82/fix view btn width (#105)

* Fix button width inconsistency

* Migrate to table style

---------

Co-authored-by: Copilot <[email protected]>
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.

AppConfig help_url default falls back to localhost instead of production URL

2 participants