Conversation
Co-authored-by: Aiden Bai <aidenybai@users.noreply.github.com>
|
Cursor Agent can help with this pull request. Just |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
@react-grab/cli
grab
@react-grab/ami
@react-grab/amp
@react-grab/claude-code
@react-grab/codex
@react-grab/copilot
@react-grab/cursor
@react-grab/droid
@react-grab/gemini
@react-grab/opencode
react-grab
@react-grab/relay
@react-grab/utils
commit: |
Co-authored-by: Aiden Bai <aidenybai@users.noreply.github.com>
Co-authored-by: Aiden Bai <aidenybai@users.noreply.github.com>
Co-authored-by: Aiden Bai <aidenybai@users.noreply.github.com>
Co-authored-by: Aiden Bai <aidenybai@users.noreply.github.com>
Co-authored-by: Aiden Bai <aidenybai@users.noreply.github.com>
|
This run croaked 😵 The workflow encountered an error before any progress could be reported. Please check the link below for details. |
| "OPENCODE", | ||
| "AMP_HOME", | ||
| "AMI", | ||
| ]; |
There was a problem hiding this comment.
Duplicated environment variable list across two packages
Low Severity
The AUTOMATION_ENVIRONMENT_VARIABLE_NAMES list in postinstall.cjs is an exact duplicate of AGENT_ENVIRONMENT_VARIABLES in packages/cli/src/utils/is-non-interactive.ts. Both enumerate the same seven environment variable names for detecting automated environments. When a new provider is added, both lists need independent manual updates, risking drift between them.
There was a problem hiding this comment.
1 issue found across 3 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="packages/website/components/install-tabs.tsx">
<violation number="1" location="packages/website/components/install-tabs.tsx:67">
P2: The CLI fallback command in the prompt loses the user's custom hotkey configuration.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
| 6. Keep all React Grab setup development-only. | ||
| 7. Show me the edited files and the exact install command you ran. | ||
| 8. If the setup does not work, suggest running the React Grab CLI: | ||
| \`npx -y grab@latest init\`.`; |
There was a problem hiding this comment.
P2: The CLI fallback command in the prompt loses the user's custom hotkey configuration.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/website/components/install-tabs.tsx, line 67:
<comment>The CLI fallback command in the prompt loses the user's custom hotkey configuration.</comment>
<file context>
@@ -39,6 +39,34 @@ const formatInitOptions = (hotkey: RecordedHotkey): string => {
+6. Keep all React Grab setup development-only.
+7. Show me the edited files and the exact install command you ran.
+8. If the setup does not work, suggest running the React Grab CLI:
+\`npx -y grab@latest init\`.`;
+};
+
</file context>
Co-authored-by: Aiden Bai <aidenybai@users.noreply.github.com>
| 6. Keep all React Grab setup development-only. | ||
| 7. Show me the edited files and the exact install command you ran. | ||
| 8. If the setup does not work, suggest running the React Grab CLI: | ||
| \`npx -y grab@latest init\`.`; |
| @@ -0,0 +1,21 @@ | |||
| const AUTOMATION_ENVIRONMENT_VARIABLE_NAMES = [ | |||
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
There was a problem hiding this comment.
1 issue found across 2 files (changes from recent commits).
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="packages/website/components/install-tabs.tsx">
<violation number="1" location="packages/website/components/install-tabs.tsx:418">
P2: The `transition-[max-height]` class will not animate because CSS cannot interpolate to/from an undefined (none) `max-height`.
When `isPromptExpanded` becomes true, the `maxHeight` inline style is removed and the container snaps to full height instantly. To animate smoothly, transition to a specific large value (like `9999px`) or just remove the transition class if snapping is acceptable.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
Co-authored-by: Aiden Bai <aidenybai@users.noreply.github.com>
Co-authored-by: Aiden Bai <aidenybai@users.noreply.github.com>



Add an environment-aware
postinstallscript toreact-grabto guide users on running theinitcommand after installation.Summary by cubic
Add a CJS
postinstallscript toreact-grabthat prints anpx -y grab@latest inithint only in CI/automated runs. Expand the website Prompt tab with a full manual setup guide (Next.js App/Pages, Vite, TanStack Start, monorepos), a collapsible prompt with smooth, auto‑measured height animation and a top-right copy button, plus a CLI fallback.Written for commit 1fda3a2. Summary will update on new commits.