log url on instant-cli errors (#2806) [bump to v1.0.55] by drew-harris · Pull Request #2806 · instantdb/instant

drew-harris · 2026-07-30T20:08:51Z

Shows the url on errors for the info command. For any InstantHttpError, the url is already part of the error message.

coderabbitai · 2026-07-30T20:09:24Z

📝 Walkthrough

Walkthrough

The CLI info command now includes the resolved base URL in dashboard fetch errors while preserving the original cause. The shared client version is updated from v1.0.54 to v1.0.55.

Changes

CLI error diagnostics

Layer / File(s)	Summary
Dashboard error URL context `client/packages/cli/src/commands/info.ts`	The info command resolves the base URL and includes it in `/dash/me` fetch errors while preserving the original cause.

Shared version update

Layer / File(s)	Summary
Version constant update `client/packages/version/src/version.ts`	The exported version constant changes from `v1.0.54` to `v1.0.55`.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

instantdb/instant#2280: Both PRs bump the shared exported version constant.
instantdb/instant#2452: Both PRs modify the shared version constant.
instantdb/instant#2524: Both PRs update the exported version value.

Suggested reviewers: nezaj, stopachka

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title matches the main change by calling out URL logging for instant-cli errors.
Description check	✅ Passed	The description is related to the info command error handling and URL inclusion in errors.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch drewh/better-logging

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

coderabbitai

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@client/packages/cli/src/commands/info.ts`:
- Around line 36-39: Reuse the existing mapError logic that adds the resolved
dashboard URL context for the later `/dash/apps/...` request as well as the
`/dash/me` request. Update the app-fetch error handling near the dashboard
request so schema and non-InstantHttpError failures are wrapped consistently,
without changing successful responses.

In `@client/packages/cli/src/layer.ts`:
- Line 87: Update the error formatting in layer.ts around the request error
handler so methodAndUrl is included only once: retain it in the structured
prefix or the InstantHttpError message, and remove the duplicate from the other
source while preserving the underlying error message.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e4261dd0-403d-4f8a-bb6f-e17cf892b3d2

📥 Commits

Reviewing files that changed from the base of the PR and between 72b2ee5 and b2d3eb6.

📒 Files selected for processing (2)

client/packages/cli/src/commands/info.ts
client/packages/cli/src/layer.ts

coderabbitai · 2026-07-30T20:11:23Z

+          (e) =>
+            new Error(`Couldn't get user information (${baseUrl}).`, {
+              cause: e,
+            }),


🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Apply URL context to the app fetch as well.

This mapError only covers /dash/me; the later /dash/apps/... request can still fail without the resolved dashboard URL, especially for schema or non-InstantHttpError failures. Reuse the same error mapping for both dashboard requests.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@client/packages/cli/src/commands/info.ts` around lines 36 - 39, Reuse the existing mapError logic that adds the resolved dashboard URL context for the later `/dash/apps/...` request as well as the `/dash/me` request. Update the app-fetch error handling near the dashboard request so schema and non-InstantHttpError failures are wrapped consistently, without changing successful responses.

the later /dash/apps request can fail but ti will fail as an InstantHttpError which will print the url.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash set -euo pipefail ast-grep outline client/packages/cli/src/layer.ts --items all printf '\n--- InstantHttpError definitions and relevant usage ---\n' rg -n -C 5 '\bInstantHttpError\b|\bInstantHTTPError\b|function printRedErrors|const printRedErrors|printRedErrors\s*=' client/packages/cli/src printf '\n--- info command request/error handling ---\n' sed -n '20,65p' client/packages/cli/src/commands/info.ts

Length of output: 9912

🏁 Script executed:

#!/bin/bash set -euo pipefail printf '%s\n' '--- InstantHttpError printer ---' sed -n '70,110p' client/packages/cli/src/layer.ts printf '%s\n' '--- HTTP client status-error transformation ---' sed -n '45,125p' client/packages/cli/src/lib/http.ts printf '%s\n' '--- Existing dashboard schema error wrappers ---' rg -n -C 4 'schemaBodyJson\(Dash(App|Me)Response\)|Couldn.t get (user|app) information' client/packages/cli/src

Length of output: 6750

@drew-harris, verified: non-2xx responses do become InstantHttpError, so that path is covered.

The remaining case is a 2xx response that fails schemaBodyJson(DashAppResponse)—for example, malformed JSON or a response that no longer matches the expected dashboard schema. That failure occurs after filterStatusOk and is not an InstantHttpError, so it will not have methodAndUrl.

So the requested wrapping is only needed for that decode-error path; the HTTP-error path is already handled as you described.

_{You are interacting with an AI system.}

github-actions · 2026-07-30T20:15:30Z

View Vercel preview at instant-www-js-drewh-better-logging-jsv.vercel.app.

stopachka

SGTM!

coderabbitai

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@client/packages/cli/src/commands/info.ts`:
- Around line 36-39: Sanitize the baseUrl value before interpolating it into the
error created in the user-information request handler. Reuse the project’s
existing URL-redaction utility if available, ensuring embedded credentials and
token-bearing query parameters are removed while retaining useful URL context;
keep the original URL for the request itself.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7366d6f1-a0c9-48e4-9fdb-b969ac2c162f

📥 Commits

Reviewing files that changed from the base of the PR and between c63d6dd and d7db68d.

📒 Files selected for processing (2)

client/packages/cli/src/commands/info.ts
client/packages/version/src/version.ts

coderabbitai · 2026-07-30T20:30:30Z

+          (e) =>
+            new Error(`Couldn't get user information (${baseUrl}).`, {
+              cause: e,
+            }),


🔒 Security & Privacy | 🟠 Major | ⚡ Quick win

Redact sensitive URL components before logging.

baseUrl may come directly from INSTANT_CLI_API_URI, so interpolating it verbatim can expose embedded credentials or token-bearing query parameters in CLI/CI logs. Log a sanitized URL instead.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@client/packages/cli/src/commands/info.ts` around lines 36 - 39, Sanitize the baseUrl value before interpolating it into the error created in the user-information request handler. Reuse the project’s existing URL-redaction utility if available, ensuring embedded credentials and token-bearing query parameters are removed while retaining useful URL context; keep the original URL for the request itself.

drew-harris temporarily deployed to Preview July 30, 2026 20:09 — with GitHub Actions Inactive

coderabbitai Bot reviewed Jul 30, 2026

View reviewed changes

drew-harris force-pushed the drewh/better-logging branch from b2d3eb6 to 0eb2c9a Compare July 30, 2026 20:13

stopachka approved these changes Jul 30, 2026

View reviewed changes

drew-harris temporarily deployed to Preview July 30, 2026 20:19 — with GitHub Actions Inactive

nezaj reviewed Jul 30, 2026

View reviewed changes

Comment thread client/packages/cli/src/commands/info.ts

nezaj reviewed Jul 30, 2026

View reviewed changes

Comment thread client/packages/version/src/version.ts Outdated

drew-harris added 2 commits July 30, 2026 13:28

log url on info error

ac6efab

bump version to v1.0.55

d7db68d

drew-harris force-pushed the drewh/better-logging branch from c63d6dd to d7db68d Compare July 30, 2026 20:28

drew-harris temporarily deployed to Preview July 30, 2026 20:28 — with GitHub Actions Inactive

drew-harris deployed to Preview July 30, 2026 20:28 — with GitHub Actions View deployment

coderabbitai Bot reviewed Jul 30, 2026

View reviewed changes

drew-harris changed the title ~~log url on instant-cli errors~~ log url on instant-cli errors (#2806) [bump to v1.0.55] Jul 30, 2026

drew-harris merged commit 44b78bb into main Jul 30, 2026
35 checks passed

drew-harris deleted the drewh/better-logging branch July 30, 2026 20:34

Navigation Menu

Uh oh!

log url on instant-cli errors (#2806) [bump to v1.0.55] - #2806

log url on instant-cli errors (#2806) [bump to v1.0.55]#2806
drew-harris merged 2 commits into
mainfrom
drewh/better-logging

drew-harris commented Jul 30, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Jul 30, 2026 •

edited

Loading

Walkthrough

Changes

Uh oh!

coderabbitai Bot left a comment

Uh oh!

coderabbitai Bot Jul 30, 2026 •

edited

Loading

Uh oh!

drew-harris Jul 30, 2026

Uh oh!

coderabbitai Bot Jul 30, 2026

Uh oh!

Uh oh!

github-actions Bot commented Jul 30, 2026

Uh oh!

stopachka left a comment

Uh oh!

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Uh oh!

coderabbitai Bot Jul 30, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

drew-harris commented Jul 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai Bot commented Jul 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jul 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

drew-harris Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions Bot commented Jul 30, 2026

Uh oh!

stopachka left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

drew-harris commented Jul 30, 2026 •

edited

Loading

coderabbitai Bot commented Jul 30, 2026 •

edited

Loading

coderabbitai Bot Jul 30, 2026 •

edited

Loading