Skip to content

Wire missing proxy exports and React hook#4

Closed
Bandonker with Codex wants to merge 2 commits into
release/v1.1.7from
codex/sub-pr-3
Closed

Wire missing proxy exports and React hook#4
Bandonker with Codex wants to merge 2 commits into
release/v1.1.7from
codex/sub-pr-3

Conversation

@Codex

@Codex Codex AI commented Feb 6, 2026

Copy link
Copy Markdown

Summary

Surface the proxy server helpers from the root entry point and add the documented useAudioUrl hook so React examples and imports resolve correctly.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring (no functional changes)
  • Performance improvement
  • Test improvement
  • Build/CI improvement

Environment Scope

  • Tauri
  • Electron
  • Web Browser
  • Node.js/Server
  • All environments

Changes Made

  • Export AudioProxyServer, createProxyServer, and startProxyServer from src/index.ts to align with server API usage.
  • Add useAudioUrl hook in src/react.ts, wrapping useAudioProxy and normalizing errors to Error for React consumers.

Example

import { startProxyServer } from 'desktop-audio-proxy';
import { useAudioUrl } from 'desktop-audio-proxy/react';

const url = await (await startProxyServer()).getProxyUrl();
const { playableUrl } = useAudioUrl('https://example.com/media.mp4');

Testing

  • Unit tests pass (npm test)
  • Integration tests pass
  • Manual testing completed
  • Tested in relevant environment(s)
  • Added new tests for new functionality

Test Environment

  • OS: Ubuntu (CI runner)
  • Node.js version: 20.x
  • Framework version (if applicable): N/A
  • Browser (if applicable): N/A

Breaking Changes

  • None

Documentation

  • README updated
  • Code comments added/updated
  • Examples updated/added
  • CHANGELOG updated
  • No documentation changes needed

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Related Issues

Additional Notes

None.

@Codex Codex AI assigned Codex and Bandonker Feb 6, 2026
@Codex Codex AI changed the title [WIP] Release v1.1.7 with video streaming and documentation updates Seek clarification on pending request Feb 6, 2026
@Codex
Codex AI requested a review from Bandonker February 6, 2026 22:24
@Bandonker

Copy link
Copy Markdown
Owner

@codex[agent] look to see if there is any junk code or something thats not wired up

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Bravo.

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

@Codex

Codex AI commented Feb 6, 2026

Copy link
Copy Markdown
Author

@codex[agent] look to see if there is any junk code or something thats not wired up

Hooked up the missing pieces: added useAudioUrl (matching docs/examples) and exported the proxy server helpers from the root entry so imports work; changes in 4922fd0.

@Codex Codex AI changed the title Seek clarification on pending request Wire missing React hook and server exports Feb 6, 2026
@Codex Codex AI changed the title Wire missing React hook and server exports Wire missing proxy exports and React hook Feb 6, 2026
@Bandonker Bandonker closed this Feb 6, 2026
@Bandonker
Bandonker deleted the codex/sub-pr-3 branch February 6, 2026 22:49
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.

2 participants