fix(logs): show --level in base44 logs --help - #589
Merged
Conversation
`--level` was registered with `.hideHelp()`, so a fully working, tested filter was invisible in `--help`. Every other `hideHelp()` in the repo marks a deprecated alias (`--org`, `--project-id`); `--level` is not one. Adds a help test that fails without the fix. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Contributor
🚀 Package Preview Available!Install this PR's preview build with npm: npm i @base44-preview/cli@0.1.8-pr.589.9d22ef2Prefer not to change any import paths? Install using npm alias so your code still imports npm i "base44@npm:@base44-preview/cli@0.1.8-pr.589.9d22ef2"Or add it to your {
"dependencies": {
"base44": "npm:@base44-preview/cli@0.1.8-pr.589.9d22ef2"
}
}
Preview published to npm registry — try new features instantly! |
davidsu
enabled auto-merge (squash)
August 6, 2026 13:12
yardend-wix
approved these changes
Aug 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
base44 logs --helpdoes not list--level, even though the flag works. It is registered with.hideHelp():How this surfaced
Our own
base44-troubleshooterskill (repobase44/skills) makes--levelthe first action of its troubleshooting flow —SKILL.md:41, under "### 1. Check Recent Errors":It appears again at
SKILL.md:49and:55, and inreferences/project-logs.mdat line 20 (options table), 36 and 57. So the skill teaches a flag that--helpsays does not exist.An agent debugging a live app (silent per-item drops in a function returning
200 {"ok":true}) hit exactly that mismatch. The reported cost is the strongest argument here: catching the skill apparently "lying" once made it stop trusting the skill's flag guidance and start guessing at CLI behaviour — which produced a false bug report (--order asc silently drops the newest entries) that had to be retracted. A working-but-hidden flag doesn't just cost a lookup; it pushes agents from reading docs to guessing, and guesses become bogus findings.Scope, verified rather than assumed: all 8 flags in the reference's option table were checked against
logs --help.--levelis the only mismatch —--function,--since,--until,-n/--limit,--order,--env,-f/--followall appear, and--jsonis correctly a global flag. (Otherlogsdefects were found in the same session —--orderbeing a no-op for a single function,--limit's documented default, log ingestion lag. Those are sibling findings tracked separately, deliberately not folded into this one-line fix.)Credit to the
logs-detectiveagent for the investigation and for correcting its own initial framing (the gap is help-text-vs-reference-docs, not "undocumented").The hide was deliberate — this PR argues for reversing it
--levelwas added and hidden in the same commit, a88ce8e (#323, @kfirstri), whose description says:Why it should be reversed anyway:
--orderand--envare equally niche and both appear in help.hideHelp()in the repo means something different — deprecated aliases (--org,--project-id/--projectId).--levelis the only hidden flag that is a live feature, so it reads like legacy.🤖 Generated by Claude), so that sentence may narrate the diff rather than record intent; the PR merged 14 min after opening with its one reviewDISMISSEDand no inline comments.@kfirstri — if the minimal-surface call was deliberately yours and you still want it, say so and I'll close this.
Fix
Drop
.hideHelp():Tests
Added
documents --level in --help, confirmed to discriminate (fails withhideHelp(), passes without).bun run test tests/cli/logs.spec.ts→ 30 passed;typecheckclean;linthas 1 pre-existing info in an unrelated fileexec.spec.tsfailures were Deno unable to reachregistry.npmjs.orgin my sandbox; CI's ubuntu+windows × npm+binary jobs all pass🤖 Generated with Claude Code