-
Notifications
You must be signed in to change notification settings - Fork 344
[codecane] feat: deploy new cli app #344
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
f630c52 to
1453f6d
Compare
…arding; align with production flow by introducing curated WELCOME_FLOW_TOPICS for onboarding and updating related UI logic to rely on new topic mappings. 🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
registry in the CLI staging workflow to ensure npm publish uses the correct registry and credentials. 🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
…f-community. Use NEXT_PUBLIC_CODEBUFF_APP_URL to resolve /api/releases/download/{version}/{fileName}; fallback to codebuff.com/api/releases/download when not configured. Generated with Codebuff. Co-Authored-By: Codebuff <noreply@codebuff.com>
|
Bun v1.3.0 (b0a6feca) Linux x64 (baseline) panic(main thread): Segmentation fault at address 0x72D5782A6248 To send a redacted crash report to Bun's team, |
981a01b to
bdd404b
Compare
|
error: Cannot find package 'web-tree-sitter' from 'B:~BUN\root\codecane.exe' Bun v1.3.1-canary.106+7662de963 (Windows x64) |
|
Bun v1.3.0 (b0a6feca) Windows x64 (baseline) panic(main thread): Segmentation fault at address 0x1EB0ED61258 To send a redacted crash report to Bun's team, B:/~BUN/root/opentui-d94x8j91.dll |
b3280d9 to
783e239
Compare
919a1f0 to
d23e975
Compare
Changes: - SDK build:types now uses dts-bundle-generator to create bundled types - Added pretypecheck hooks to npm-app and evals packages - This ensures SDK types are properly bundled at dist/index.d.ts Benefits: - Single bundled type file instead of nested structure - Works correctly with SDK's package.json types field - Packages can resolve @codebuff/sdk types during typecheck
When the SDK is bundled and called from other packages during tests, the ripgrep binary path resolution was failing because: 1. Caller's import.meta.url was passed to getBundledRgPath() 2. SDK tried to resolve vendor/ relative to the caller's location 3. This failed in workspace monorepo setups Changes: - SDK now uses its own import.meta.url as fallback when none provided - Added distPath check for bundled SDK (dist/index.mjs → dist/vendor/) - npm-app now calls getBundledRgPath() without passing import.meta.url - Added pretest hook to build SDK before running npm-app tests Fixes npm-app test failures where ripgrep binary couldn't be found.
Resolved conflicts in: - cli/src/hooks/use-send-message.ts - cli/src/state/chat-store.ts
The API endpoint changed from agent_run_id to run_id in origin/main. Updated test expectations to match the new parameter names and error messages. Changes: - Updated request bodies: agent_run_id → run_id - Updated error messages: agentRunId → runId - Updated mock function parameter: agentRunId → runId
Test fixes to match updated API contracts and property names: - code-map: queryText → queryPathOrContent - billing: Updated mock implementations for async module loading - openrouter-ai-sdk: Fixed streaming test mocks for ReadableStream handling These test updates were previously in commit 119218a but were reverted. Re-applying only the test fixes without other changes from that commit.
The Playwright test was causing errors when running bun test at the root level since Playwright tests need to be run separately with playwright test command.
The --cwd flag requires '=' syntax when used with 'bun run' in bun 1.3.0. Without it, bun displays help menu instead of executing the command, causing SDK type generation to be skipped during CI typecheck.
The test jobs run individual test files with 'bun test <file>', which bypasses the package.json pretest hook. This caused npm-app tests to fail with 'Cannot find module @codebuff/sdk' because the SDK wasn't built. Changes: - Added 'Build SDK before tests' step in test job - Added 'Build SDK before integration tests' step in test-integration job - This ensures SDK dist artifacts (index.mjs, index.cjs, vendor/) exist before tests that depend on @codebuff/sdk are executed Root cause: CI test command bypasses npm hooks by running test files directly.
No description provided.