Skip to content

feat: user client.response.text turn from frontend#6

Merged
HananAlsawy merged 6 commits intomainfrom
feat-user-client.response.text-turn-from-frontend
Oct 6, 2025
Merged

feat: user client.response.text turn from frontend#6
HananAlsawy merged 6 commits intomainfrom
feat-user-client.response.text-turn-from-frontend

Conversation

@HananAlsawy
Copy link
Copy Markdown
Contributor

@HananAlsawy to run npm i before marking the PR ready for review

@HananAlsawy HananAlsawy marked this pull request as ready for review October 6, 2025 11:48
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

Comment thread package-lock.json Outdated
Comment on lines +27 to +33
"../layercode-js-sdk": {
"name": "@layercode/js-sdk",
"version": "2.2.0",
"license": "MIT",
"dependencies": {
"@ricky0123/vad-web": "^0.0.24",
"onnxruntime-web": "^1.21.1"
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 Remove local file reference for @layercode/js-sdk in lock file

The lockfile now resolves @layercode/js-sdk from ../layercode-js-sdk and marks it as a linked package. Anyone who runs npm install in a fresh clone will fail unless they have that sibling directory available, whereas the previous lockfile pulled the dependency from the registry. This breaks reproducible installs and blocks builds for other developers. Regenerate package-lock.json so the dependency points to the published 2.2.0 package instead of a local path.

Useful? React with 👍 / 👎.

@HananAlsawy
Copy link
Copy Markdown
Contributor Author

@codex please review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Delightful!

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

@HananAlsawy HananAlsawy merged commit 3bfad74 into main Oct 6, 2025
2 checks passed
@HananAlsawy HananAlsawy deleted the feat-user-client.response.text-turn-from-frontend branch October 6, 2025 12:01
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