Merge pull request #33 from redhat-developer/fix-konflux-tekton-updates-skill

fix: skill should not hardcode 1- in the...

Author: nickboldt

skills/konflux-tekton-updates/SKILL.md

@@ -91,3 +91,4 @@ Use live `MIGRATION.md` as source of truth. Common cases:
 - Leaving removed task params (`dev-package-managers`, `COMMIT_SHA` on `build-image-index`).
 - Adding `verify_*` guards that fail on the next Konflux bump.
 - Dropping `image-expires-after` from PLRs only because `build-image-index` no longer uses it.
+- Hardcoding `1-` in `generatePipelineRunsForPlugins.sh` Containerfile comments; use `${RHDH_XY_VERSION}` so `1.10.0` becomes `1-10`, not `1`.

skills/konflux-tekton-updates/references/plugin-catalog.md

@@ -31,3 +31,22 @@ Add to the PipelineRun heredoc `spec.params` when `oci-plugin-build-pipeline` ga
 ```
 
 Do not embed full `pipelineSpec` in the generator.
+
+## Version naming (x.y.z → x-y)
+
+`generatePipelineRunsForPlugins.sh` derives `RHDH_XY_VERSION` from `-v x.y.z` (e.g. `1.10.0` → `1-10`). Use it everywhere; never hardcode `1-` in generated paths.
+
+| Pattern | Example for 1.10.0 |
+|---------|-------------------|
+| PLR / Containerfile basename | `bcp-rbac-1-10-push.{yaml,Containerfile}` |
+| Konflux component / application | `bcp-rbac-1-10`, `rhdh-plugin-catalog-1-10` |
+| Target branch in CEL | `rhdh-1.10-rhel-9` (dots, not dashes) |
+| Containerfile builder comment | `.tekton/plugin-catalog-builder-1-10-push.yaml` |
+
+After regenerate, grep for stale `-1-push` (without minor version):
+
+```bash
+rg '-1-push' .tekton --glob '*.Containerfile'
+```
+
+A hit like `plugin-catalog-builder-1-push.yaml` means the generator heredoc still hardcodes `1-` instead of `${RHDH_XY_VERSION}`.