Skip to content

feat(runner-context): Update runner-context schema#2926

Merged
javirln merged 2 commits into
chainloop-dev:mainfrom
javirln:feat/pfm-5142-schema
Mar 23, 2026
Merged

feat(runner-context): Update runner-context schema#2926
javirln merged 2 commits into
chainloop-dev:mainfrom
javirln:feat/pfm-5142-schema

Conversation

@javirln

@javirln javirln commented Mar 23, 2026

Copy link
Copy Markdown
Member

Update the runner context JSON schema to include the new tags field, aligning with the gatherer internal changes.

The schema now accepts tag protection configurations alongside branches, enabling validation of runner context data that includes tag ruleset information from GitHub and GitLab.

Signed-off-by: Javier Rodriguez <javier@chainloop.dev>

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

1 issue found across 1 file

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="internal/schemavalidators/internal_schemas/runnercontext/runner-context-response-0.1.schema.json">

<violation number="1" location="internal/schemavalidators/internal_schemas/runnercontext/runner-context-response-0.1.schema.json:268">
P2: Add a runner-context test case that exercises `data.repos[].tags`; otherwise this new schema path can regress without detection.

(Based on your team's feedback about adding or updating tests for new paths.) [FEEDBACK_USED]</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Signed-off-by: Javier Rodriguez <javier@chainloop.dev>
@javirln javirln merged commit a778549 into chainloop-dev:main Mar 23, 2026
15 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