Skip to content

🚸 Add skills field to UI - #55

Open
taagarwa-rh wants to merge 1 commit into
mainfrom
fix/job-ui-missing-skills
Open

taagarwa-rh wants to merge 1 commit into
mainfrom
fix/job-ui-missing-skills

Conversation

@taagarwa-rh

Copy link
Copy Markdown
Collaborator

No description provided.

@coderabbitai

coderabbitai Bot commented Sep 18, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: b2a5b828-14c2-47ca-8390-400f8cb19928


Comment @coderabbitai help to get the list of available commands.

@rounakbende10

Copy link
Copy Markdown
Collaborator

Just one finding otherwise the skills submission logic otherwise looks correct.
The field advertises local paths that the queue API rejects
Location: ui.py:301–302 [https://github.com/redhat-et/coding_agent_bench/blob/4d7b1e23c3dee01f1d948440668b6ab8a995867b/src/coding_agent_bench/ui.py#L301-L302]
The placeholder suggests ./skills/review, and the help text says paths inside task containers are supported. However, this form submits to /jobs, which only accepts Git skill sources.

Following the displayed example produces:
HTTP 400: Remote skill source './skills/review' is not a Git source.

Suggested fix: describe the field as comma-separated Git sources, using examples such as:
obra/superpowers, org/repo@main
Remove the local-path example and task-container-path guidance.

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