Skip to content

Document replay timestamp formatting - #15

Open
ben-million wants to merge 1 commit into
mainfrom
cursor/basic-code-comment
Open

Document replay timestamp formatting#15
ben-million wants to merge 1 commit into
mainfrom
cursor/basic-code-comment

Conversation

@ben-million

Copy link
Copy Markdown
Collaborator

Summary

  • Add a brief comment explaining the compact replay timestamp display format.

Test plan

  • Not run (comment-only change).

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented May 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
skills Ready Ready Preview, Comment May 10, 2026 3:43am

@react-doctor

react-doctor Bot commented May 10, 2026

Copy link
Copy Markdown

React Review Report

React Review couldn't analyze this pull request because it couldn't find a package.json at the repository root.

This usually happens when the React app lives in a subdirectory, such as in a monorepo. For now, React Review expects the project it reviews to be at the repository root.

Technical details
Sandbox command failed: node /vercel/sandbox/review-runtime/review-runner.mjs
file:///vercel/sandbox/review-runtime/node_modules/react-doctor/dist/index.js:945
	if (!isFile(packageJsonPath)) throw new Error(`No package.json found in ${directory}`);
	                                    ^

Error: No package.json found in /vercel/sandbox/review-workspaces/head/repo
    at discoverProject (file:///vercel/sandbox/review-runtime/node_modules/react-doctor/dist/index.js:945:38)
    at diagnose (file:///vercel/sandbox/review-runtime/node_modules/react-doctor/dist/index.js:3085:22)
    at file:///vercel/sandbox/review-runtime/review-runner.mjs:21:3

Node.js v24.14.1

@ben-million

ben-million commented May 10, 2026

Copy link
Copy Markdown
Collaborator Author

88 score

-2.2% vs base

❌ 0 errors (0)
⚠️ 25 warnings (+2)

Merging this PR reduces React health by 2.2%

Prompt to copy to agent

@ben-million

ben-million commented May 10, 2026

Copy link
Copy Markdown
Collaborator Author

88 score

-2.2% vs base

0 errors (0)
⚠️ 25 warnings (+2)

Warning

Merging this PR reduces React health by 2.2%

Prompt to copy to agent
Run react-doctor and fix the React issues in this PR.

React health score: 88 (-2 (-2.2%) vs base)
Errors: 0 (0 vs base)
Warnings: 25 (+2 vs base)

New issues:
- apps/website/app/copy-prompt/copy-prompt-client.tsx:52 [warning] js-batch-dom-css: Multiple sequential element.style assignments — batch with cssText or classList for fewer reflows
  Help: Batch DOM/CSS reads and writes — interleaving them inside a loop causes layout thrashing. Read first, then write
- apps/website/app/copy-prompt/copy-prompt-client.tsx:86 [warning] no-cascading-set-state: 5 setState calls in a single useEffect — consider using useReducer or deriving state
  Help: Combine into useReducer: `const [state, dispatch] = useReducer(reducer, initialState)`

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