lc dev: backlog fixes, dep modernization, CI + release

[dp9u0]

Summary

Batch of work accumulated on dev since the last merge to main:

Bug fixes (from upstream backlog triage + local incidents)

• problems.json cache: 24h TTL + purge on every login method (stale-cache mass-desync incident)
• checkError: cloudflare 403/429 no longer misreported as session expired
• parseCookie crash on invalid cookie; log.js toString crash on null args
• getUserInfo -> userStatus query (fixes premium/locked + "login as undefined")
• stat -c ERR_OUT_OF_RANGE on modern node; -v/-vv silently exiting (yargs argv snapshot)
• output columns sized to data (4-digit ids, long names)

Features

• lc show -d daily challenge; official hints in show + generated files
• problem URL accepted as keyword (show/test/submit)
• tag filtering actually works now (-t stack), fetched via graphql
• plain-text problem descriptions in lc show
• non-zero exit codes on failures; solved markers in lc cache

Dependency modernization

• dropped: sqlite3 (unused), lodash, wordwrap, mkdirp, prompt (-> readline helper)
• request (deprecated) -> axios-backed adapter (lib/http.js)
• eslint 5 -> 10 (flat config), mocha 11, nock 14, cheerio 1.2, underscore 1.13.8
• audit: 55 findings (2 critical) -> 10, none on the runtime path

Engineering

• security: user.json chmod 0600 (holds a live session cookie)
• CI: lint+test on node 20/22/24; release workflow builds @yao-pkg/pkg binaries + checksums and publishes a GitHub Release on merge to main
• test suite: 171 -> 185 (new command-layer coverage)

Merging this to main will trigger the first binary release (v3.0.0 tag).

🤖 Generated with Claude Code

[Copilot]

🟡 Changes recommended

It introduces at least one confirmed runtime-breaking bug in the new HTTP shim integration (and other issues) that should be fixed before merge.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

This PR is a large “dev backlog” merge that modernizes the project’s dependencies and CI/release pipeline while adding several LeetCode feature fixes (daily challenge, tags, hints, cache TTL/purging) and replacing the deprecated request dependency with an axios-backed adapter.

Changes:

• Replaced request (and other legacy deps) with an axios-backed HTTP shim and updated plugins/commands accordingly.
• Added/extended CLI functionality and resiliency (daily challenge, hints/tag support, better cache semantics, non-zero exit codes on failures).
• Modernized engineering workflow (ESLint flat config, expanded unit tests, CI matrix, automated release workflow + pkg targets).

File summaries

File	Description
test/test_plugin.js	Updates expected install command string for plugin deps.
test/test_log.js	Adds coverage for null/undefined log args and exit code behavior.
test/test_file.js	Prevents env var leakage across test suites.
test/test_core.js	Adds tests for URL keywords and daily challenge resolution.
test/plugins/test_retry.js	Adds coverage for skipping relogin without saved password.
test/plugins/test_leetcode.js	Adds extensive coverage for cookie login, error mapping, tags, hints, daily challenge.
test/plugins/test_cache.js	Adds coverage for problems cache TTL/meta + purge-on-login behavior.
test/mock/find-the-difference.json.20171216	Updates fixture to include hints and normalized JSON formatting.
test/commands/test_user.js	Adds command-layer tests for user output/logout/error exit codes.
test/commands/test_submission.js	Adds command-layer tests for submission download and error exit codes.
test/commands/test_stat.js	Adds command-layer tests for progress/graph rendering.
test/commands/test_show.js	Adds command-layer tests for plain-text rendering + hints + codegen.
test/commands/test_list.js	Adds command-layer tests for listing/filtering and warnings.
test/commands/test_cache.js	Adds command-layer tests for solved markers and deletion.
templates/detailed.tpl	Emits hints into generated detailed templates.
package.json	Updates dependencies/devDependencies and expands test script paths; pkg targets/output.
lib/session.js	Adds BLOCKED error and chmod hardening for persisted session cookie file.
lib/plugins/solution.discuss.js	Switches HTTP client to new adapter and updates solution URL path.
lib/plugins/retry.js	Avoids burning retries when no saved password is available.
lib/plugins/leetcode.js	Migrates off request, adds tags fetch, daily challenge, hints, improved error mapping, cookie parsing changes.
lib/plugins/leetcode.cn.js	Migrates off request to new adapter.
lib/plugins/cache.js	Adds problems TTL/meta versioning, stale fallback, and purges caches on all login methods.
lib/plugin.js	Uses new HTTP adapter; changes plugin dep install command; refactors download/copy logic.
lib/log.js	Sets process.exitCode on failures; makes logging tolerant of null/undefined args.
lib/http.js	Adds axios-backed drop-in request shim with redirect and jar support.
lib/helper.js	Adds problemsMeta key, startCase helper, and readInput prompt replacement.
lib/file.js	Replaces mkdirp with fs.mkdirSync(..., {recursive:true}).
lib/core.js	Accepts full problem URLs as keywords; adds daily challenge flow; replaces wordwrap dependency.
lib/commands/version.js	Uses optional catch binding cleanup.
lib/commands/user.js	Replaces prompt with readline-based input and avoids “login as undefined”.
lib/commands/test.js	Replaces lodash.startCase; sets exit code on failed tests.
lib/commands/submit.js	Replaces lodash.startCase; sets exit code on failed submits.
lib/commands/submission.js	Sets exit code when per-problem task fails.
lib/commands/stat.js	Fixes width alignment + Buffer.write out-of-range issue.
lib/commands/show.js	Adds daily option, plain-text description output, and hint printing.
lib/commands/session.js	Removes prompt dependency (cleanup).
lib/commands/list.js	Makes output columns dynamic to data widths.
lib/commands/config.js	Uses optional catch binding cleanup.
lib/commands/cache.js	Widens columns dynamically and adds solved markers for per-problem caches.
lib/cli.js	Routes TRACE HTTP debugging to new adapter; fixes yargs -v/-vv handling.
eslint.config.js	Adds ESLint flat config equivalent to previous ruleset.
docs/releases.md	Removes legacy Jekyll release notes page content.
docs/install.md	Removes legacy Jekyll installation page content.
docs/index.html	Removes legacy Jekyll docs landing page content.
docs/demo.html	Removes legacy Jekyll demo page content.
docs/commands.md	Removes legacy Jekyll command docs content.
docs/advanced.md	Removes legacy Jekyll advanced tips content.
docs/_config.yml	Removes Jekyll theme config.
bin/leetcode	Removes process.emitWarning suppression wrapper.
1.js	Adds a sample solution file (likely accidental/temporary).
.github/workflows/release.yml	Adds release workflow to build pkg binaries and publish GitHub Releases.
.github/workflows/ci.yml	Adds CI workflow for lint/test on Node 20/22/24.
.eslintrc.js	Removes legacy ESLint config in favor of flat config.

Review details

Suppressed comments (1)

lib/plugin.js:73

• Switching plugin dependency installation to pnpm add introduces a new runtime requirement: end users who installed leetcode-cli via npm (or run the packaged binaries) are unlikely to have pnpm available on PATH, so plugin installs may start failing. Consider using npm install --save (always present with Node) or adding a fallback/detection layer.

• Files reviewed: 52/57 changed files
• Comments generated: 3
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.