Skip to content

Uncontrolled data used in path expression #70 - #44

Merged
manit6752025 merged 3 commits into
mainfrom
Uncontrolled-data-used-in-path-expression-#70
Oct 20, 2026
Merged

manit6752025 merged 3 commits into
mainfrom
Uncontrolled-data-used-in-path-expression-#70

Conversation

@manit6752025

Copy link
Copy Markdown
Collaborator

AXIOM Pull Request

Summary

Adds an API Playground page to the AXIOM Control Center, making local API endpoints easier to discover through a dedicated interface.

Why

The API Playground E2E test could not find the expected “Choose an endpoint” heading because the api-playground page was not registered in the application routing. This change adds the missing page and connects it to the workspace navigation.

What changed

  • Added an ApiPlayground React component.
  • Registered the api-playground page in the application page type and route handling.
  • Added API Playground navigation and page metadata.
  • Added a bounded catalogue of eight endpoint references, including health, models, datasets, runtime, events, diagnostics, workflows, and evaluations.
  • Kept the interface read-only: selecting an endpoint changes the reference view without sending an API request or modifying workspace data.

Testing

Commands:
- npm run build — passed in the previously shared run, before the latest route changes.
- npm run e2e — 14 passed, 1 failed in the previously shared run; the API Playground route was missing at that time.
- npx playwright test e2e/api-playground.spec.ts — rerun after the route fix.
- git diff --check — not yet verified.

UI changes

  • UI not affected
  • UI manually tested
  • Screenshots included where useful
  • Responsive behaviour checked where relevant

Documentation

  • No documentation changes required
  • README updated
  • Wiki updated
  • Other documentation updated

Release / packaging impact

  • No release impact
  • Release workflow changed
  • Windows packaging affected
  • Linux packaging affected
  • Release notes should mention this change

Security

  • No security-sensitive changes
  • Security implications reviewed
  • No secrets or credentials included

Breaking changes

  • No breaking changes
  • Breaking change described below

If applicable, describe migration or compatibility requirements:

None expected. The change adds a frontend page without changing existing API contracts.

Checklist

  • Change is focused
  • Existing project conventions followed
  • Tests completed
  • git diff --check passes
  • Documentation updated where needed
  • No passwords, tokens, private keys or other secrets included
  • Known limitations are documented
  • Commit history is understandable

Additional context

The endpoint catalogue is a reference interface, not an interactive API client. Endpoint existence and backend compatibility should be verified against the running AXIOM API. Re-run the build and E2E tests after applying the route fix before merging.


AXIOM — Build AI. Own AI.

@manit6752025 manit6752025 added this to the CWE FIX milestone Oct 20, 2026
@manit6752025 manit6752025 self-assigned this Oct 20, 2026
@manit6752025 manit6752025 added the bug Something isn't working label Oct 20, 2026
@manit6752025 manit6752025 linked an issue Oct 20, 2026 that may be closed by this pull request
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-10-20T09:48:41.515561Z 896d71a PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@manit6752025
manit6752025 merged commit fde0495 into main Oct 20, 2026
5 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Uncontrolled data used in path expression #70

1 participant