Skip to content

Add agentic workflow context files#2890

Merged
tekton-robot merged 1 commit into
tektoncd:mainfrom
ngelman1:agent-ready
Jun 8, 2026
Merged

Add agentic workflow context files#2890
tekton-robot merged 1 commit into
tektoncd:mainfrom
ngelman1:agent-ready

Conversation

@ngelman1

@ngelman1 ngelman1 commented May 28, 2026

Copy link
Copy Markdown
Contributor

Prepare the Tekton CLI repository for agentic workflows as part of SRVKP-11822.

  • AGENTS.md: Agent-oriented context file covering build/test commands, key conventions (Cobra command tree, golden-file tests, vendored deps, version injection via ldflags), architecture overview, PR conventions, and skill triggers

  • CLAUDE.md: Redirect to AGENTS.md (Claude Code convention)

  • .claude/skills/: Commit-message and release-notes skills adapted from tektoncd/operator PR #3387, with all review feedback incorporated:

  • Makefile: Add PKG variable support to lint-go and test-unit targets to allow running tests on a single package

Agentready Overall Score: 63.3/100 (Silver)

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

  • Includes tests (if functionality changed/added)
  • Run the code checkers with make check
  • Regenerate the manpages, docs and go formatting with make generated
  • Commit messages follow commit message best practices

See the contribution guide
for more details.

Release Notes

@tekton-robot tekton-robot added the release-note Denotes a PR that will be considered when it comes time to generate release notes. label May 28, 2026
@tekton-robot tekton-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label May 28, 2026
@divyansh42

Copy link
Copy Markdown
Member

/retest

@divyansh42 divyansh42 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for adding the changes @ngelman1
Overall changes look good. I see PR tektoncd/operator#3387 got updated and has plenty of reviews by the community. Can you please consider those reviews as well and incorporate in this PR?

Comment thread .agents/skills Outdated
@@ -0,0 +1 @@
../.claude/skills No newline at end of file

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we please add skills to .agents and symlink in .claude.
I am just thinking to make it generic

@divyansh42 divyansh42 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Changes looks good.
Just a few minor comments

Comment thread .agents/skills/commit-message/SKILL.md Outdated
Comment thread .agents/skills/commit-message/SKILL.md Outdated
Comment thread .agents/skills/commit-message/SKILL.md Outdated
Comment thread .agents/skills/commit-message/SKILL.md Outdated
@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 4, 2026
@divyansh42

Copy link
Copy Markdown
Member

/retest

1 similar comment
@divyansh42

Copy link
Copy Markdown
Member

/retest

@ngelman1

ngelman1 commented Jun 7, 2026

Copy link
Copy Markdown
Contributor Author

@divyansh42 I see that hte same e2e tests are failing again.. but I see they the same tests fail in multiple PRs so I guess its not a problem with my changes?

@divyansh42

Copy link
Copy Markdown
Member

/retest

@divyansh42

Copy link
Copy Markdown
Member

/cc @pratap0007

@vdemeester vdemeester left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The failure are due to 1.13.x, the entrypoint binary is logging WARN messages about termination message parsing to stderr, and those are being captured in the logs, and it fails.. either we need to wait and bump to 1.13.1 (due this week), or we need to use 1.12.x for a while

@tekton-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: divyansh42, vdemeester

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [divyansh42,vdemeester]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@vdemeester

Copy link
Copy Markdown
Member

The main branch is also red/broken because of the same issue. An another alternative is to make sure having this extra output doesn't make the build fail..

@pratap0007

Copy link
Copy Markdown
Contributor

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 8, 2026
@vdemeester

Copy link
Copy Markdown
Member

@ngelman1 this will need to be rebased after #2897 gets merged.

@tekton-robot tekton-robot removed the lgtm Indicates that a PR is ready to be merged. label Jun 8, 2026
@vdemeester

Copy link
Copy Markdown
Member

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 8, 2026
@vdemeester

Copy link
Copy Markdown
Member

/retest

@tekton-robot tekton-robot merged commit 9e8e184 into tektoncd:main Jun 8, 2026
18 of 20 checks passed
@ngelman1 ngelman1 deleted the agent-ready branch June 8, 2026 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants