Repository navigation
fix: log in with the configured username - #15
Conversation
The login request hard-coded user_id=zhuchenrui2 since the initial commit, so anyone else's credentials were rejected while the CLI still printed "Logged in as <their username>". Send config.username (URL-encoded) instead. Also align the version across the CLI, package.json and deno.json (previously v1.5.1 / 1.5.1 / 1.5.3) at 1.5.4. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JhQLfJB3LWW4vkNXK85xL1
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e71bf182ae
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "jsdom": "^23.0.1" | ||
| }, | ||
| "version": "1.5.1", | ||
| "version": "1.5.4", |
There was a problem hiding this comment.
Regenerate package-lock for the 1.5.4 bump
The tracked package-lock.json still declares the root package as version 1.5.1 in both of its root metadata fields. With this package.json bump, running npm install immediately rewrites those fields to 1.5.4, leaving contributors with a dirty worktree and tools that inspect the lockfile reporting stale release metadata; update the lockfile as part of this version change.
Useful? React with 👍 / 👎.
Summary
index.tshard-codeduser_id=zhuchenrui2(since the initial commit,2fdb606), ignoringusernamefromquicksubmit.json. For any other user, XMOJ returned "UserName or Password Wrong!" while the CLI still printedLogged in as <configured username>.encodeURIComponent(config.username)..version()in the CLI,package.jsonanddeno.json(they were v1.5.1 / 1.5.1 / 1.5.3).Validation
deno lint index.tspasses.deno install && deno compile -A index.tssucceeds; the binary reportsv1.5.4.zhuchenrui2account before publishing.deno fmt --checkalready fails onmainfor unrelated lines, so the file was not reformatted. Only the changed line was wrapped to Deno style.🤖 Generated with Claude Code
https://claude.ai/code/session_01JhQLfJB3LWW4vkNXK85xL1