Skip to content

feat: disable web UI by default#172

Merged
jdx merged 1 commit intomainfrom
feat/disable-web-ui-default
Jan 19, 2026
Merged

feat: disable web UI by default#172
jdx merged 1 commit intomainfrom
feat/disable-web-ui-default

Conversation

@jdx
Copy link
Owner

@jdx jdx commented Jan 19, 2026

Summary

The web UI is now opt-in rather than always running. To enable it, specify a port via --web-port or PITCHFORK_WEB_PORT environment variable:

# Via environment variable
export PITCHFORK_WEB_PORT=19876
pitchfork supervisor start --force

# Via command line
pitchfork supervisor run --web-port 19876

Rationale

  • Reduces resource usage for users who only use the CLI or TUI
  • The TUI now has feature parity with the web UI (including config editing)
  • Simplifies the default setup

Changes

  • Removed --no-web flag (no longer needed)
  • Made --web-port optional with no default value
  • Updated docs to reflect that web UI must be explicitly enabled

🤖 Generated with Claude Code


Note

feat: web UI is now opt-in

  • CLI changes: --web-port now enables the web UI when provided (no default); removed --no-web. supervisor run reads optional PITCHFORK_WEB_PORT and passes Option<u16> to SUPERVISOR.start.
  • Docs updated: guidance across web-ui.md, logs.md, quickstart, env vars, and generated CLI docs to reflect opt-in behavior and enablement via --web-port or PITCHFORK_WEB_PORT.

Written by Cursor Bugbot for commit 2fa0e2c. This will update automatically on new commits. Configure here.

The web UI is now opt-in rather than always running. To enable it,
specify a port via `--web-port` or `PITCHFORK_WEB_PORT` environment
variable.

This reduces resource usage for users who only use the CLI or TUI.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings January 19, 2026 16:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR changes the web UI from enabled-by-default to opt-in, requiring explicit port configuration via --web-port or PITCHFORK_WEB_PORT environment variable. This reduces resource usage for users who only need CLI/TUI functionality.

Changes:

  • Removed the --no-web flag and made --web-port an optional parameter with no default value
  • Updated all documentation to reflect that the web UI must be explicitly enabled
  • Simplified the supervisor run logic by removing the conditional web port handling

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/cli/supervisor/run.rs Changed web_port from u16 with default to Option<u16>, removed no_web flag and conditional logic
pitchfork.usage.kdl Updated help text to indicate web UI is opt-in, removed --no-web flag and default port value
docs/reference/environment-vars.md Updated to document that web UI is disabled by default, removed PITCHFORK_NO_WEB section
docs/quickstart.md Removed web UI references from quick start, focusing on TUI as the primary interactive dashboard
docs/guides/web-ui.md Restructured to show how to enable the web UI first, removed port configuration and disable sections
docs/guides/logs.md Updated to mention both TUI and web UI, with web UI noted as optional
docs/cli/supervisor/run.md Updated --web-port description to reflect opt-in behavior, removed --no-web documentation
docs/cli/commands.json Updated help text for --web-port flag and removed --no-web entry with default value

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jdx jdx merged commit 3c7efa8 into main Jan 19, 2026
4 checks passed
@jdx jdx deleted the feat/disable-web-ui-default branch January 19, 2026 16:44
@jdx jdx mentioned this pull request Jan 19, 2026
jdx added a commit that referenced this pull request Jan 19, 2026
## 🤖 New release

* `pitchfork-cli`: 1.0.2 -> 1.1.0

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

## [1.1.0](v1.0.2...v1.1.0) -
2026-01-19

### Added

- add file watching to auto-restart daemons
([#165](#165))
- support boolean values for retry configuration
([#170](#170))
- disable web UI by default
([#172](#172))
- auto-generate JSON schema from Rust types
([#167](#167))

### Fixed

- improve cron watcher granularity for sub-minute schedules
([#163](#163))
- improve log file position tracking accuracy
([#164](#164))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Releases `pitchfork-cli` v1.1.0 and syncs version across `Cargo.toml`,
`Cargo.lock`, `docs/cli/*`, and `pitchfork.usage.kdl`.
> 
> - **Added**: file watching to auto-restart daemons; boolean support
for retry config; web UI disabled by default; JSON schema generation
from Rust types
> - **Fixed**: improved cron watcher granularity (sub-minute); more
accurate log file position tracking
> - Updated `CHANGELOG.md` with 1.1.0 notes
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
2e6d4c6. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
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.

2 participants