Skip to content

feat: OPE46-00048: QoL: Add thresholds override flags - #16

Open
andrei-hasna wants to merge 1 commit into
mainfrom
factory/eedf4ac7-ec9d-4fdc-9b33-0a1c5f09-ac07f4df
Open

feat: OPE46-00048: QoL: Add thresholds override flags#16
andrei-hasna wants to merge 1 commit into
mainfrom
factory/eedf4ac7-ec9d-4fdc-9b33-0a1c5f09-ac07f4df

Conversation

@andrei-hasna

@andrei-hasna andrei-hasna commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Objective

OPE46-00048: QoL: Add thresholds override flags

Add threshold override flags: monitor doctor --cpu-threshold 80 --mem-threshold 85 for one-off threshold adjustments.

MEASURED on origin/main: monitor doctor [machine] (src/cli/index.ts:522) has only --limit/--cursor/--verbose/--json; thresholds come from config. Add one-off override flags --cpu-threshold and --mem-threshold (and any other thresholds the doctor already reads) that take precedence over config for that invocation only.

EXECUTION NOTES (added 2026-07-29 for autonomous execution)
Repo: github.com/hasna/monitor. CLI commands are registered in src/cli/index.ts (Commander). Verified against origin/main that this surface does NOT already exist.
Requirements: implement in src/ (not docs); keep existing output shapes and the -j/--json contract intact; reuse the existing helpers in that file (parseLimitOption, parseCursorOption, chalk formatting) rather than inventing new patterns.
Acceptance: bun run typecheck, bun run build and bun test are all green, and a new/extended test under src/ exercises the new behaviour (assert on the command's JSON output and/or exit code, not on colored text). No network, no live remote machines, no credentials — tests must run offline against local collectors/fixtures.

Verification

  • policy source: base c4cbd1e (immutable commit — agent-proof)
  • ⚠️ GATE-INTEGRITY: agent touched verify-bearing config — REVIEW (scripts.typecheck)
  • containment: env — allowlist env, non-login shell, run-scoped HOME (registry auth seeded for install)
  • install: pass
  • typecheck: pass
  • lint: pass
  • build: pass
  • test: pass
  • doctor (ci): ok — 11 checks passed (1 advisory)

Run run_7d6202cce403 · backend codewith · task eedf4ac7-ec9d-4fdc-9b33-0a1c5f09a54d
🏭 Generated by @hasnaxyz/factory


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

OPE46-00048: QoL: Add thresholds override flags

Add threshold override flags: `monitor doctor --cpu-threshold 80 --mem-threshold 85` for one-off threshold adjustments.

MEASURED on origin/main: `monitor doctor [machine]` (src/cli/index.ts:522) has only --limit/--cursor/--verbose/--json; thresholds come from config. Add one-off override flags --cpu-threshold <n> and --mem-threshold <n> (and any other thresholds the doctor already reads) that take precedence over config for that invocation only.

EXECUTION NOTES (added 2026-07-29 for autonomous execution)
Repo: github.com/hasna/monitor. CLI commands are registered in src/cli/index.ts (Commander). Verified against origin/main that this surface does NOT already exist.
Requirements: implement in src/ (not docs); keep existing output shapes and the -j/--json contract intact; reuse the existing helpers in that file (parseLimitOption, parseCursorOption, chalk formatting) rather than inventing new patterns.
Acceptance: `bun run typecheck`, `bun run build` and `bun test` are all green, and a new/extended test under src/ exercises the new behaviour (assert on the command's JSON output and/or exit code, not on colored text). No network, no live remote machines, no credentials — tests must run offline against local collectors/fixtures.

X-Factory-Run: run_7d6202cce403
X-Factory-Task: eedf4ac7-ec9d-4fdc-9b33-0a1c5f09a54d
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