Skip to content

removing unused components#1678

Merged
maknop merged 1 commit into
mainfrom
remove_unused_components_tekton
Jun 11, 2026
Merged

removing unused components#1678
maknop merged 1 commit into
mainfrom
remove_unused_components_tekton

Conversation

@maknop

@maknop maknop commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Given recent architectural decisions, the following has been removed as tekton components:

  • backend
  • frontend
  • operator
  • public API

Summary by CodeRabbit

  • Chores
    • Removed automated pipeline configuration files for backend, frontend, operator, and public API components. These are internal infrastructure changes with no impact to user-facing functionality.

@netlify

netlify Bot commented Jun 11, 2026

Copy link
Copy Markdown

Deploy Preview for cheerful-kitten-f556a0 canceled.

Name Link
🔨 Latest commit 2f5273b
🔍 Latest deploy log https://app.netlify.com/projects/cheerful-kitten-f556a0/deploys/6a2b0c1b211a7a000801691c

@coderabbitai

coderabbitai Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: c5e1f21b-2fd0-4cb4-bd08-7217f58ae544

📥 Commits

Reviewing files that changed from the base of the PR and between f4e07df and 2f5273b.

📒 Files selected for processing (8)
  • .tekton/ambient-code-backend-main-pull-request.yaml
  • .tekton/ambient-code-backend-main-push.yaml
  • .tekton/ambient-code-frontend-main-pull-request.yaml
  • .tekton/ambient-code-frontend-main-push.yaml
  • .tekton/ambient-code-operator-main-pull-request.yaml
  • .tekton/ambient-code-operator-main-push.yaml
  • .tekton/ambient-code-public-api-main-pull-request.yaml
  • .tekton/ambient-code-public-api-main-push.yaml
💤 Files with no reviewable changes (8)
  • .tekton/ambient-code-public-api-main-pull-request.yaml
  • .tekton/ambient-code-frontend-main-push.yaml
  • .tekton/ambient-code-backend-main-pull-request.yaml
  • .tekton/ambient-code-operator-main-pull-request.yaml
  • .tekton/ambient-code-backend-main-push.yaml
  • .tekton/ambient-code-frontend-main-pull-request.yaml
  • .tekton/ambient-code-operator-main-push.yaml
  • .tekton/ambient-code-public-api-main-push.yaml

📝 Walkthrough

Walkthrough

This PR removes eight Tekton PipelineRun manifests (4,684 lines total) that previously orchestrated container image builds, security scans, and artifact pushes for backend, frontend, operator, and public-api components on both pull-request and push events to the main branch.

Changes

Cohort / File(s) Summary
Tekton Pipeline Removal
.tekton/ambient-code-backend-main-pull-request.yaml, .tekton/ambient-code-backend-main-push.yaml, .tekton/ambient-code-frontend-main-pull-request.yaml, .tekton/ambient-code-frontend-main-push.yaml, .tekton/ambient-code-operator-main-pull-request.yaml, .tekton/ambient-code-operator-main-push.yaml, .tekton/ambient-code-public-api-main-pull-request.yaml, .tekton/ambient-code-public-api-main-push.yaml
Removes all PipelineRun manifests that performed image builds, security checks (Clair, SAST, ClamAV, Coverity), tagging, and artifact push for four components. Each component had separate PR and push pipelines triggered on source changes or pipeline YAML modifications. All pipelines used shared build service accounts, workspaces (git-auth, netrc), and conditional skip-checks gates.

Possibly related PRs

  • ambient-code/platform#1644: Adds the Tekton PipelineRun manifests for the same four components and workflows that this PR deletes.

Suggested labels

ambient-code:self-reviewed

🚥 Pre-merge checks | ✅ 7 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title does not follow Conventional Commits format (type(scope): description) and lacks a specific type prefix required by the guidelines. Reformat to Conventional Commits: e.g., 'chore: remove unused Tekton pipeline components' or 'ci: remove obsolete Tekton manifests'.
✅ Passed checks (7 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Performance And Algorithmic Complexity ✅ Passed PR appears to delete Tekton PipelineRun YAMLs; in current repo the mentioned .tekton files don’t exist, and no code changes indicate added O(n^2)/N+1/loop-expensive work.
Security And Secret Handling ✅ Passed PR deletes Tekton PipelineRun YAMLs (backend/frontend/operator/public-api files absent in .tekton). Remaining .tekton YAMLs contain no hardcoded token/password-like strings.
Kubernetes Resource Safety ✅ Passed All deleted Tekton PipelineRun manifests (.tekton/ambient-code-*-main-{push,pull}.yaml) are absent in the repo; no Jobs/Secrets/PVCs or RBAC/limit issues are introduced by these deletions.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch remove_unused_components_tekton
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch remove_unused_components_tekton

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@maknop maknop merged commit 338c379 into main Jun 11, 2026
18 checks passed
@maknop maknop deleted the remove_unused_components_tekton branch June 11, 2026 19:32
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.

1 participant