Skip to content

fix(comms): OpenFangAPI.baseUrl is undefined#1006

Merged
jaberjaber23 merged 1 commit intoRightNow-AI:mainfrom
lc-soft:fix/comms-api-base-url
Apr 10, 2026
Merged

fix(comms): OpenFangAPI.baseUrl is undefined#1006
jaberjaber23 merged 1 commit intoRightNow-AI:mainfrom
lc-soft:fix/comms-api-base-url

Conversation

@lc-soft
Copy link
Copy Markdown
Contributor

@lc-soft lc-soft commented Apr 7, 2026

Summary

Fixed an issue where the stream API URL in the Agent Comms page contained "undefined". This was caused by the OpenFangAPI object missing a baseUrl property, resulting in malformed URLs when constructing API endpoints.

Changes

OpenFangAPI.baseUrl deleted

Before:

image

After:

image

Testing

  • cargo clippy --workspace --all-targets -- -D warnings passes
  • cargo test --workspace passes
  • Live integration tested (if applicable)

Security

  • No new unsafe code
  • No secrets or API keys in diff
  • User input validated at boundaries

@lc-soft lc-soft closed this Apr 8, 2026
@lc-soft lc-soft deleted the fix/comms-api-base-url branch April 8, 2026 06:36
@lc-soft lc-soft restored the fix/comms-api-base-url branch April 8, 2026 06:36
@lc-soft lc-soft reopened this Apr 10, 2026
Copy link
Copy Markdown
Member

@jaberjaber23 jaberjaber23 left a comment

Choose a reason for hiding this comment

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

LGTM. OpenFangAPI.baseUrl is undefined, causing 'undefined/api/...' URLs. Using relative path is correct.

@jaberjaber23 jaberjaber23 merged commit 2ef5704 into RightNow-AI:main Apr 10, 2026
11 of 22 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.

2 participants