Skip to content

SRVKP-12189: Remove TektonHub integration from pipelines task catalog and quick search#1095

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift-pipelines:mainfrom
anwesha-palit-redhat:feat/SRVKP-12189
Jun 22, 2026
Merged

SRVKP-12189: Remove TektonHub integration from pipelines task catalog and quick search#1095
openshift-merge-bot[bot] merged 1 commit into
openshift-pipelines:mainfrom
anwesha-palit-redhat:feat/SRVKP-12189

Conversation

@anwesha-palit-redhat

@anwesha-palit-redhat anwesha-palit-redhat commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Removes TektonHub from the pipelines console plugin and consolidates task catalog behavior on ArtifactHub. Also hardens ArtifactHub quick search install/update flows, fixes enable-devconsole-integration gating, and prevents installing/updating the wrong YAML when the user switches versions.

Changes

TektonHub removal

  • Removed TektonHub API, provider, types, tests, and console extension metadata.
  • Removed TektonHub-specific quick search logic from PipelineQuickSearch, PipelineQuickSearchDetails, and related utils.
  • Updated catalog extensions and Cypress feature files for ArtifactHub-only behavior.

ArtifactHub quick search hardening

  • CTA race fix: In PipelineQuickSearch, fetch ArtifactHub task details on-demand when selectedVersionContentUrl is missing before create/update.
  • Version mismatch fix: Track selectedVersionForContentUrl alongside the cached URL; getSelectedVersionUrl returns null when the cached URL does not match the requested version, forcing a refetch for the correct YAML. >>> this is from Qodo
  • Version switch UX: Disable CTA while version details are loading in PipelineQuickSearchDetails.
  • Defensive checks: Reject invalid/missing URLs in createArtifactHubTask / updateArtifactHubTask.
  • Error logging: Log update failures at both call sites (PipelineQuickSearch, useArtifactHubTasksProvider).

enable-devconsole-integration fix

TektonConfig commands

Enable ArtifactHub dev console integration (default when param is absent):

oc patch tektonconfig config --type=merge -p '{"spec":{"hub":{"params":[{"name":"enable-devconsole-integration","value":"true"}]}}}'

Disable ArtifactHub dev console integration:

oc patch tektonconfig config --type=merge -p '{"spec":{"hub":{"params":[{"name":"enable-devconsole-integration","value":"false"}]}}}'

Verify current value:

oc get tektonconfig config -o jsonpath='{.spec.hub.params}' | jq .

After patching, hard refresh the console (Cmd+Shift+R / Ctrl+Shift+R) so the plugin re-reads TektonConfig.

Screen recordings

  • TektonHub deprecation: Open Pipeline Builder quick search; confirm no TektonHub tasks/provider labels and no network calls to hub.tekton.dev
Screen.Recording.2026-06-18.at.12.56.53.mov
  • ArtifactHub enabled: With enable-devconsole-integration=true, search in Pipeline Builder; confirm ArtifactHub tasks appear, can be installed and updated and ensure requests go to artifacthub.io....
part.-.1.mov
part-2.mov
  • ArtifactHub disabled: Patch param to "false", hard refresh, search again; confirm no ArtifactHub API calls and no ArtifactHub tasks in results (both initial catalog load and live search).
Screen.Recording.2026-06-18.at.12.39.15.mov

@openshift-ci openshift-ci Bot requested review from vdemeester and vikram-raj June 3, 2026 06:36
@openshift-ci openshift-ci Bot added the approved Label for Approved PRs label Jun 3, 2026
@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor Author

/agentic_review

@qodo-code-review

This comment was marked as outdated.

Comment thread src/components/task-quicksearch/PipelineQuickSearch.tsx
@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor Author

/agentic_describe

@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor Author

/agentic_review

@anwesha-palit-redhat anwesha-palit-redhat removed the approved Label for Approved PRs label Jun 16, 2026
@qodo-code-review

qodo-code-review Bot commented Jun 16, 2026

Copy link
Copy Markdown

Code review by qodo was updated up to the latest commit ba8f317

@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Remove TektonHub integration from pipelines task catalog and quick search
✨ Enhancement 🧪 Tests ⚙️ Configuration changes 🕐 40+ Minutes

Grey Divider

Description

• Removes TektonHub CRD/model and task provider integration from the console plugin.
• Standardizes “Community” task catalog to use Artifact Hub and updates quick-search behaviors.
• Updates UI strings and test coverage to reflect the Artifact Hub-only workflow.
Diagram

graph TD
  A["console-extensions.json"] --> B["Catalog extensions"] --> C["ArtifactHubTasksProvider"] --> D["useHubIntegration"] --> E[("TektonConfig CR")]
  C --> F{{"Artifact Hub API"}}
  G["Pipeline Quick Search UI"] --> B
  subgraph Legend
    direction LR
    _cfg["Config/Extensions"] ~~~ _ui["UI Component"] ~~~ _svc["Provider/Hook"] ~~~ _db[("K8s Resource")] ~~~ _ext{{"External"}}
  end
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Deprecate TektonHub behind a feature flag
  • ➕ Reduces immediate behavioral change risk for existing clusters
  • ➕ Allows phased rollout and easier rollback if regressions appear
  • ➖ Keeps dead code paths, tests, and translations around longer
  • ➖ Ongoing maintenance burden for an already-removed integration
2. Introduce a generic "ExternalHub" provider abstraction
  • ➕ Makes adding/removing future providers less invasive
  • ➕ Centralizes CTA/version/installed behaviors for external sources
  • ➖ More upfront refactor than needed if Artifact Hub is the only target
  • ➖ Adds abstraction complexity without immediate functional benefit

Recommendation: The PR’s approach (complete removal of TektonHub wiring and tests) is appropriate if TektonHub support is officially discontinued. It meaningfully reduces surface area (models/providers/quick-search branches) and aligns catalog UX around Artifact Hub. Only consider the feature-flag deprecation alternative if downstream consumers still rely on TektonHub behaviors and need a transition window.

Files changed (20) +127 / -529

Enhancement (1) +7 / -36
PipelineQuickSearch.tsxRemove TektonHub-specific install/upgrade flow branches +7/-36

Remove TektonHub-specific install/upgrade flow branches

• Narrows installed-version decoration to Artifact Hub items only and falls back to existing installed metadata when versions are missing. Removes the TektonHub code path that created/updated Tasks via raw URL fetches, and adds a guard to avoid proceeding when the selected version URL is unavailable.

src/components/task-quicksearch/PipelineQuickSearch.tsx

Refactor (8) +25 / -184
catalog-utils.tsRename hub integration hook and remove TektonHub platform filtering helpers +2/-12

Rename hub integration hook and remove TektonHub platform filtering helpers

• Removes TektonHub-specific platform filtering utilities and the exported integration key constant usage. Renames the integration hook to a provider-agnostic 'useHubIntegration' while still reading the 'enable-devconsole-integration' param from TektonConfig.

src/components/catalog/catalog-utils.ts

index.tsStop exporting TektonHub task provider +0/-2

Stop exporting TektonHub task provider

• Removes the export for 'TektonHubTaskProvider', leaving Artifact Hub and in-cluster providers.

src/components/catalog/providers/index.ts

useArtifactHubTasksProvider.tsxSwitch integration gating to useHubIntegration and simplify typing +3/-6

Switch integration gating to useHubIntegration and simplify typing

• Replaces 'useTektonHubIntegration' usage with 'useHubIntegration' for gating external hub integration. Removes TektonHubTask-specific generic typing from the normalized CatalogItem list.

src/components/catalog/providers/useArtifactHubTasksProvider.tsx

PipelineQuickSearchDetails.tsxEliminate TektonHub version fetching and hub link UI +8/-61

Eliminate TektonHub version fetching and hub link UI

• Removes TektonHub-specific version loading (via hub API) and the “Read more” external link to hub UI. Fixes effect cleanup by returning an unmount handler and restricts details fetching to Artifact Hub items.

src/components/task-quicksearch/PipelineQuickSearchDetails.tsx

PipelineQuickSearchVersionDropdown.tsxDecouple version dropdown typing from TektonHub types +6/-2

Decouple version dropdown typing from TektonHub types

• Replaces 'TektonHubTaskVersion' dependency with a lightweight 'TaskVersion' shape used by Artifact Hub versions. Keeps the dropdown behavior unchanged while making it provider-agnostic.

src/components/task-quicksearch/PipelineQuickSearchVersionDropdown.tsx

const.tsRemove TEKTONHUB constant +0/-1

Remove TEKTONHUB constant

• Drops the TektonHub constant, leaving ArtifactHub as the only external source identifier.

src/components/task-quicksearch/const.ts

pipeline-quicksearch-utils.tsRemove TektonHub provider, annotations, and task create/update helpers +6/-83

Remove TektonHub provider, annotations, and task create/update helpers

• Eliminates TektonHub provider support and the create/update Task helpers that fetched and applied YAML content. Simplifies installed-from annotation handling to Artifact Hub only, and makes selected version URL resolution depend solely on 'selectedVersionContentUrl'.

src/components/task-quicksearch/pipeline-quicksearch-utils.ts

models.tsRemove TektonHubModel from K8s model registry +0/-17

Remove TektonHubModel from K8s model registry

• Deletes the TektonHub K8s model definition, removing plugin-level awareness of the TektonHub CRD.

src/models.ts

Tests (4) +93 / -265
create-from-builder-page.featureRemove TektonHub-based pipeline builder scenarios +0/-52

Remove TektonHub-based pipeline builder scenarios

• Deletes regression/manual scenarios that validate installing/upgrading TektonHub tasks and disabling Tektonhub integration from TektonConfig. Keeps remaining pipeline builder scenarios intact.

integration-tests/cypress/features/pipelines/create-from-builder-page.feature

PipelineQuicksearchDetails.spec.tsxRewrite quick-search details tests to target Artifact Hub behavior +24/-155

Rewrite quick-search details tests to target Artifact Hub behavior

• Removes TektonHub-specific mocks and cases (consoleFetch version API, hub link, and repeated version fetching). Updates fixtures to use Artifact Hub sample items and adjusts CTA expectations accordingly.

src/components/task-quicksearch/tests/PipelineQuicksearchDetails.spec.tsx

catalog-item-data.tsRename TektonHub catalog fixtures to Artifact Hub fixtures +15/-16

Rename TektonHub catalog fixtures to Artifact Hub fixtures

• Renames sample data types and exported fixtures from TektonHub-oriented naming to ArtifactHub equivalents. Loosens sample task typing to generic records to avoid TektonHub type dependencies.

src/components/task-quicksearch/tests/catalog-item-data.ts

pipeline-quicksearch-utils.spec.tsUpdate pipeline quick-search util tests for Artifact Hub-only semantics +54/-42

Update pipeline quick-search util tests for Artifact Hub-only semantics

• Updates assertions and fixtures to use Artifact Hub items and annotations. Adjusts getSelectedVersionUrl expectations to rely on 'selectedVersionContentUrl' and removes TektonHub-only test cases.

src/components/task-quicksearch/tests/pipeline-quicksearch-utils.spec.ts

Documentation (6) +1 / -14
plugin__pipelines-console-plugin.jsonUpdate Community catalog copy and remove TektonHub strings +1/-4

Update Community catalog copy and remove TektonHub strings

• Adds a new string for “Browse community tasks.” and removes TektonHub-related translations, including the “Read more” label and TektonHub/TektonHubs resource labels.

locales/en/plugin__pipelines-console-plugin.json

plugin__pipelines-console-plugin.jsonRemove TektonHub/TektonHubs locale keys (es) +0/-2

Remove TektonHub/TektonHubs locale keys (es)

• Deletes TektonHub and TektonHubs translation entries from the Spanish locale file.

locales/es/plugin__pipelines-console-plugin.json

plugin__pipelines-console-plugin.jsonRemove TektonHub/TektonHubs locale keys (fr) +0/-2

Remove TektonHub/TektonHubs locale keys (fr)

• Deletes TektonHub and TektonHubs translation entries from the French locale file.

locales/fr/plugin__pipelines-console-plugin.json

plugin__pipelines-console-plugin.jsonRemove TektonHub/TektonHubs locale keys (ja) +0/-2

Remove TektonHub/TektonHubs locale keys (ja)

• Deletes TektonHub and TektonHubs translation entries from the Japanese locale file.

locales/ja/plugin__pipelines-console-plugin.json

plugin__pipelines-console-plugin.jsonRemove TektonHub/TektonHubs locale keys (ko) +0/-2

Remove TektonHub/TektonHubs locale keys (ko)

• Deletes TektonHub and TektonHubs translation entries from the Korean locale file.

locales/ko/plugin__pipelines-console-plugin.json

plugin__pipelines-console-plugin.jsonRemove TektonHub/TektonHubs locale keys (zh) +0/-2

Remove TektonHub/TektonHubs locale keys (zh)

• Deletes TektonHub and TektonHubs translation entries from the Chinese locale file.

locales/zh/plugin__pipelines-console-plugin.json

Other (1) +1 / -30
console-extensions.jsonDrop TektonHub model metadata and task provider catalog entry +1/-30

Drop TektonHub model metadata and task provider catalog entry

• Removes the TektonHub console model-metadata extension and the catalog item-provider that referenced the TektonHub task provider. Updates the Community catalog description to no longer mention Tekton Hub explicitly.

console-extensions.json

@anwesha-palit-redhat anwesha-palit-redhat changed the title Test PR - main SRVKP-12189: Remove TektonHub integration from pipelines task catalog and quick search Jun 16, 2026
@openshift-ci-robot

openshift-ci-robot commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

@anwesha-palit-redhat: This pull request references SRVKP-12189 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "5.0.0" version, but no target version was set.

Details

In response to this:

PR Summary by Qodo

Remove TektonHub integration from pipelines task catalog and quick search
✨ Enhancement 🧪 Tests ⚙️ Configuration changes 🕐 40+ Minutes

Grey Divider
Description

• Removes TektonHub CRD/model and task provider integration from the console plugin.
• Standardizes “Community” task catalog to use Artifact Hub and updates quick-search behaviors.
• Updates UI strings and test coverage to reflect the Artifact Hub-only workflow.
Diagram

graph TD
 A["console-extensions.json"] --> B["Catalog extensions"] --> C["ArtifactHubTasksProvider"] --> D["useHubIntegration"] --> E[("TektonConfig CR")]
 C --> F{{"Artifact Hub API"}}
 G["Pipeline Quick Search UI"] --> B
 subgraph Legend
   direction LR
   _cfg["Config/Extensions"] ~~~ _ui["UI Component"] ~~~ _svc["Provider/Hook"] ~~~ _db[("K8s Resource")] ~~~ _ext{{"External"}}
 end
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Deprecate TektonHub behind a feature flag
  • ➕ Reduces immediate behavioral change risk for existing clusters
  • ➕ Allows phased rollout and easier rollback if regressions appear
  • ➖ Keeps dead code paths, tests, and translations around longer
  • ➖ Ongoing maintenance burden for an already-removed integration
2. Introduce a generic "ExternalHub" provider abstraction
  • ➕ Makes adding/removing future providers less invasive
  • ➕ Centralizes CTA/version/installed behaviors for external sources
  • ➖ More upfront refactor than needed if Artifact Hub is the only target
  • ➖ Adds abstraction complexity without immediate functional benefit

Recommendation: The PR’s approach (complete removal of TektonHub wiring and tests) is appropriate if TektonHub support is officially discontinued. It meaningfully reduces surface area (models/providers/quick-search branches) and aligns catalog UX around Artifact Hub. Only consider the feature-flag deprecation alternative if downstream consumers still rely on TektonHub behaviors and need a transition window.

Files changed (20) +127 / -529

Enhancement (1) +7 / -36
PipelineQuickSearch.tsxRemove TektonHub-specific install/upgrade flow branches +7/-36

Remove TektonHub-specific install/upgrade flow branches

• Narrows installed-version decoration to Artifact Hub items only and falls back to existing installed metadata when versions are missing. Removes the TektonHub code path that created/updated Tasks via raw URL fetches, and adds a guard to avoid proceeding when the selected version URL is unavailable.

src/components/task-quicksearch/PipelineQuickSearch.tsx

Refactor (8) +25 / -184
catalog-utils.tsRename hub integration hook and remove TektonHub platform filtering helpers +2/-12

Rename hub integration hook and remove TektonHub platform filtering helpers

• Removes TektonHub-specific platform filtering utilities and the exported integration key constant usage. Renames the integration hook to a provider-agnostic 'useHubIntegration' while still reading the 'enable-devconsole-integration' param from TektonConfig.

src/components/catalog/catalog-utils.ts

index.tsStop exporting TektonHub task provider +0/-2

Stop exporting TektonHub task provider

• Removes the export for 'TektonHubTaskProvider', leaving Artifact Hub and in-cluster providers.

src/components/catalog/providers/index.ts

useArtifactHubTasksProvider.tsxSwitch integration gating to useHubIntegration and simplify typing +3/-6

Switch integration gating to useHubIntegration and simplify typing

• Replaces 'useTektonHubIntegration' usage with 'useHubIntegration' for gating external hub integration. Removes TektonHubTask-specific generic typing from the normalized CatalogItem list.

src/components/catalog/providers/useArtifactHubTasksProvider.tsx

PipelineQuickSearchDetails.tsxEliminate TektonHub version fetching and hub link UI +8/-61

Eliminate TektonHub version fetching and hub link UI

• Removes TektonHub-specific version loading (via hub API) and the “Read more” external link to hub UI. Fixes effect cleanup by returning an unmount handler and restricts details fetching to Artifact Hub items.

src/components/task-quicksearch/PipelineQuickSearchDetails.tsx

PipelineQuickSearchVersionDropdown.tsxDecouple version dropdown typing from TektonHub types +6/-2

Decouple version dropdown typing from TektonHub types

• Replaces 'TektonHubTaskVersion' dependency with a lightweight 'TaskVersion' shape used by Artifact Hub versions. Keeps the dropdown behavior unchanged while making it provider-agnostic.

src/components/task-quicksearch/PipelineQuickSearchVersionDropdown.tsx

const.tsRemove TEKTONHUB constant +0/-1

Remove TEKTONHUB constant

• Drops the TektonHub constant, leaving ArtifactHub as the only external source identifier.

src/components/task-quicksearch/const.ts

pipeline-quicksearch-utils.tsRemove TektonHub provider, annotations, and task create/update helpers +6/-83

Remove TektonHub provider, annotations, and task create/update helpers

• Eliminates TektonHub provider support and the create/update Task helpers that fetched and applied YAML content. Simplifies installed-from annotation handling to Artifact Hub only, and makes selected version URL resolution depend solely on 'selectedVersionContentUrl'.

src/components/task-quicksearch/pipeline-quicksearch-utils.ts

models.tsRemove TektonHubModel from K8s model registry +0/-17

Remove TektonHubModel from K8s model registry

• Deletes the TektonHub K8s model definition, removing plugin-level awareness of the TektonHub CRD.

src/models.ts

Tests (4) +93 / -265
create-from-builder-page.featureRemove TektonHub-based pipeline builder scenarios +0/-52

Remove TektonHub-based pipeline builder scenarios

• Deletes regression/manual scenarios that validate installing/upgrading TektonHub tasks and disabling Tektonhub integration from TektonConfig. Keeps remaining pipeline builder scenarios intact.

integration-tests/cypress/features/pipelines/create-from-builder-page.feature

PipelineQuicksearchDetails.spec.tsxRewrite quick-search details tests to target Artifact Hub behavior +24/-155

Rewrite quick-search details tests to target Artifact Hub behavior

• Removes TektonHub-specific mocks and cases (consoleFetch version API, hub link, and repeated version fetching). Updates fixtures to use Artifact Hub sample items and adjusts CTA expectations accordingly.

src/components/task-quicksearch/tests/PipelineQuicksearchDetails.spec.tsx

catalog-item-data.tsRename TektonHub catalog fixtures to Artifact Hub fixtures +15/-16

Rename TektonHub catalog fixtures to Artifact Hub fixtures

• Renames sample data types and exported fixtures from TektonHub-oriented naming to ArtifactHub equivalents. Loosens sample task typing to generic records to avoid TektonHub type dependencies.

src/components/task-quicksearch/tests/catalog-item-data.ts

pipeline-quicksearch-utils.spec.tsUpdate pipeline quick-search util tests for Artifact Hub-only semantics +54/-42

Update pipeline quick-search util tests for Artifact Hub-only semantics

• Updates assertions and fixtures to use Artifact Hub items and annotations. Adjusts getSelectedVersionUrl expectations to rely on 'selectedVersionContentUrl' and removes TektonHub-only test cases.

src/components/task-quicksearch/tests/pipeline-quicksearch-utils.spec.ts

Documentation (6) +1 / -14
plugin__pipelines-console-plugin.jsonUpdate Community catalog copy and remove TektonHub strings +1/-4

Update Community catalog copy and remove TektonHub strings

• Adds a new string for “Browse community tasks.” and removes TektonHub-related translations, including the “Read more” label and TektonHub/TektonHubs resource labels.

locales/en/plugin__pipelines-console-plugin.json

plugin__pipelines-console-plugin.jsonRemove TektonHub/TektonHubs locale keys (es) +0/-2

Remove TektonHub/TektonHubs locale keys (es)

• Deletes TektonHub and TektonHubs translation entries from the Spanish locale file.

locales/es/plugin__pipelines-console-plugin.json

plugin__pipelines-console-plugin.jsonRemove TektonHub/TektonHubs locale keys (fr) +0/-2

Remove TektonHub/TektonHubs locale keys (fr)

• Deletes TektonHub and TektonHubs translation entries from the French locale file.

locales/fr/plugin__pipelines-console-plugin.json

plugin__pipelines-console-plugin.jsonRemove TektonHub/TektonHubs locale keys (ja) +0/-2

Remove TektonHub/TektonHubs locale keys (ja)

• Deletes TektonHub and TektonHubs translation entries from the Japanese locale file.

locales/ja/plugin__pipelines-console-plugin.json

plugin__pipelines-console-plugin.jsonRemove TektonHub/TektonHubs locale keys (ko) +0/-2

Remove TektonHub/TektonHubs locale keys (ko)

• Deletes TektonHub and TektonHubs translation entries from the Korean locale file.

locales/ko/plugin__pipelines-console-plugin.json

plugin__pipelines-console-plugin.jsonRemove TektonHub/TektonHubs locale keys (zh) +0/-2

Remove TektonHub/TektonHubs locale keys (zh)

• Deletes TektonHub and TektonHubs translation entries from the Chinese locale file.

locales/zh/plugin__pipelines-console-plugin.json

Other (1) +1 / -30
console-extensions.jsonDrop TektonHub model metadata and task provider catalog entry +1/-30

Drop TektonHub model metadata and task provider catalog entry

• Removes the TektonHub console model-metadata extension and the catalog item-provider that referenced the TektonHub task provider. Updates the Community catalog description to no longer mention Tekton Hub explicitly.

console-extensions.json

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

Comment thread src/components/catalog/catalog-utils.ts Outdated
@openshift-ci openshift-ci Bot added the approved Label for Approved PRs label Jun 16, 2026
@anwesha-palit-redhat anwesha-palit-redhat force-pushed the feat/SRVKP-12189 branch 2 times, most recently from 4b5a75f to a478756 Compare June 18, 2026 06:20
@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor Author

/agentic_review

Comment thread src/components/task-quicksearch/PipelineQuickSearch.tsx
@qodo-code-review

Copy link
Copy Markdown

Code review by qodo was updated up to the latest commit a478756

Comment thread src/components/pipeline-builder/PipelineBuilderForm.tsx
@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor Author

/agentic_review

@qodo-code-review

Copy link
Copy Markdown

Code review by qodo was updated up to the latest commit 0fe6ee3

@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor Author

/agentic_review

@qodo-code-review

Copy link
Copy Markdown

Code review by qodo was updated up to the latest commit c120759

item.content_url;
selectedItem.tags = item.keywords;
if (mounted) {
selectedItem.attributes.versions = item.available_versions;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

moved these inside if block to prevent stale updates
selectedItem.attributes.versions = item.available_versions; selectedItem.attributes.selectedVersionContentUrl = item.content_url; selectedItem.tags = item.keywords;

and selectedVersionForContentUrl to track which version the cached URL belongs to - this is from Qodo Reviews

@openshift-ci openshift-ci Bot added the approved Label for Approved PRs label Jun 18, 2026
@anwesha-palit-redhat anwesha-palit-redhat removed the approved Label for Approved PRs label Jun 18, 2026
@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor Author

/agentic-review

Comment thread src/components/task-quicksearch/__tests__/catalog-item-data.ts
Comment thread src/components/catalog/catalog-utils.ts Outdated
Comment thread src/components/quick-search/utils/quick-search-utils.tsx
@openshift-ci openshift-ci Bot added the approved Label for Approved PRs label Jun 22, 2026
@anwesha-palit-redhat anwesha-palit-redhat removed the approved Label for Approved PRs label Jun 22, 2026
@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor Author

/test e2e

@qodo-code-review

qodo-code-review Bot commented Jun 22, 2026

Copy link
Copy Markdown

PR-Agent: could not find a component named e2e in a supported language in this PR.

@arvindk-softwaredev

Copy link
Copy Markdown
Contributor

verified with PR image in openshift cluster

Screen.Recording.2026-06-19.at.6.44.20.PM-1080p.mov

@arvindk-softwaredev arvindk-softwaredev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Looks Good to Me Label label Jun 22, 2026
@openshift-ci

openshift-ci Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: anwesha-palit-redhat, arvindk-softwaredev

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 [anwesha-palit-redhat,arvindk-softwaredev]

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

@openshift-ci openshift-ci Bot added the approved Label for Approved PRs label Jun 22, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit c439824 into openshift-pipelines:main Jun 22, 2026
6 checks passed
@anwesha-palit-redhat

Copy link
Copy Markdown
Contributor Author

/cherry-pick release-v1.22.x

@openshift-cherrypick-robot

Copy link
Copy Markdown

@anwesha-palit-redhat: new pull request created: #1137

Details

In response to this:

/cherry-pick release-v1.22.x

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Label for Approved PRs jira/valid-reference lgtm Looks Good to Me Label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants