Skip to content

add mic switching#14

Merged
jackbridger merged 2 commits intomainfrom
feature/mic-switching
Nov 14, 2025
Merged

add mic switching#14
jackbridger merged 2 commits intomainfrom
feature/mic-switching

Conversation

@jackbridger
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ 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".

Comment thread src/index.tsx Outdated
Comment on lines +1 to +8
import React, { useState, useEffect, useCallback, useRef } from 'react';
import LayercodeClient, {
type AgentConfig,
type AuthorizeSessionRequest,
type LayercodeAudioInputDevice,
listAudioInputDevices,
watchAudioInputDevices,
} from 'layercode-js-sdk';
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P0 Badge Import JS SDK from uninstalled module

The new hook imports LayercodeClient and helpers from 'layercode-js-sdk', but package.json only declares a dependency on @layercode/js-sdk and the README still references that scoped package. Because no module named layercode-js-sdk is installed, npm run build or consumers compiling the package will fail with “Cannot find module 'layercode-js-sdk'”. The import path should remain @layercode/js-sdk (or the dependency name should be updated consistently).

Useful? React with 👍 / 👎.

@jackbridger jackbridger force-pushed the feature/mic-switching branch from adc3931 to 03f2b76 Compare November 14, 2025 15:09
@jackbridger jackbridger force-pushed the feature/mic-switching branch from 0c8edb3 to 3fea619 Compare November 14, 2025 15:17
@jackbridger jackbridger merged commit 63f931a into main Nov 14, 2025
3 checks passed
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