17131 achieva agents

[luancazarine]

Resolves #17131

Summary by CodeRabbit

• New Features
  • Get Product by Barcode action (supports exact or partial with wildcard).
  • Batch Barcode Lookup action to retrieve multiple products at once.
  • Search Products by Parameters with filters (MPN, ASIN, title, category, manufacturer, brand, geo, text) and max results.
• Refactor
  • More reliable API requests and built-in pagination for larger result sets.
  • Clearer success summaries after execution.
• Chores
  • Package version bumped to 0.1.0.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments

Project	Deployment	Preview	Comments	Updated (UTC)
pipedream-docs	Ignored			Oct 6, 2025 5:28pm
pipedream-docs-redirect-do-not-edit	Ignored			Oct 6, 2025 5:28pm

[coderabbitai]

Walkthrough

Adds three new actions for Barcode Lookup (single barcode, batch barcodes, parameterized search), introduces a refactored app client with axios-based requests and pagination, adds a utility for parsing inputs, and updates package metadata and dependencies.

Changes

Cohort / File(s)	Summary
Actions — Barcode lookups components/barcode_lookup/actions/get-product-by-barcode/get-product-by-barcode.mjs, components/barcode_lookup/actions/batch-barcode-lookup/batch-barcode-lookup.mjs, components/barcode_lookup/actions/search-products-by-parameters/search-products-by-parameters.mjs	New actions to retrieve products: by single barcode, by multiple barcodes (joined), and by various parameters with pagination via app methods. Each exports metadata, props, and a run method delegating to app search.
App core — HTTP and pagination components/barcode_lookup/barcode_lookup.app.mjs	Replaces prior auth helper with axios-based request helpers: _baseUrl, _params, _makeRequest, searchProducts, and an async paginate generator. Removes authKeys().
Utilities components/barcode_lookup/common/utils.mjs	Adds parseObject(obj) to normalize inputs, attempting JSON.parse on strings/array elements with safe fallbacks.
Package metadata components/barcode_lookup/package.json	Bumps version to 0.1.0 and adds dependency on @pipedream/platform ^3.1.0.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant User
  participant Action as Search by Parameters Action
  participant App as barcode_lookup.app
  participant API as BarcodeLookup API

  User->>Action: Run with props (mpn/asin/title/... , maxResults)
  Action->>App: paginate({ fn: searchProducts, params })
  loop Pages until done/maxResults
    App->>API: GET /products?params&api_key=...
    API-->>App: { products, total/has_more, page }
    App-->>Action: yield each product
  end
  Action-->>User: Array of products (aggregated)
  Note over Action,App: Summary set with count

Loading

sequenceDiagram
  autonumber
  participant User
  participant Action as Batch Barcode Lookup
  participant Util as utils.parseObject
  participant App as barcode_lookup.app
  participant API as BarcodeLookup API

  User->>Action: Run with props (barcodes[])
  Action->>Util: parseObject(barcodes)
  Util-->>Action: normalized array
  Action->>App: searchProducts({ params: { barcode: "a,b,c" } })
  App->>API: GET /products?barcode=a,b,c&api_key=...
  API-->>App: Products
  App-->>Action: Response
  Action-->>User: Response (+ summary)

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Poem

I thump my paws on fields of code,
Barcodes bloom where data flowed.
One hop, batch hop—search I go,
Pages turn with api_key in tow.
A nibble of utils, axios breeze—
Products found with rabbit ease.
0.1.0—carrots, please! 🥕

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings, 1 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues Check	⚠️ Warning	The changes add BarcodeLookup actions and utility methods but do not implement support for uploading or reading Excel files nor the creation of Shopify products from retrieved barcodes as specified in issue #17131.	Implement the missing functionality to upload and parse Excel files and to use the retrieved product data to create products on the Shopify website per the linked issue requirements.
Description Check	⚠️ Warning	The pull request description only includes a reference to the issue and does not follow the repository template, lacking the required "## WHY" section or any explanation of the rationale for the changes.	Please expand the description to use the provided template by adding the "## WHY" section and details explaining the motivation behind these changes and how they address the linked issue.
Title Check	❓ Inconclusive	The title "17131 achieva agents" is vague and only references the issue number and project name without describing the actual changes made, so it does not clearly summarize the primary purpose of the pull request.	Please update the title to clearly and concisely describe the primary change introduced by this pull request, for example "Add BarcodeLookup actions for Achieva Agents integration".

✅ Passed checks (2 passed)

Check name	Status	Explanation
Out of Scope Changes Check	✅ Passed	All code additions and modifications focus on integrating BarcodeLookup API actions and related utilities, which aligns with the linked issue’s goal of Achieva Agents integration without introducing unrelated functionality.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch 17131-achieva-agents

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 3

🧹 Nitpick comments (1)

components/barcode_lookup/actions/search-products-by-parameters/search-products-by-parameters.mjs (1)

97-100: Consider simplifying async iterator collection.

The manual array building is correct but could be more concise. While not required, consider this alternative if you prefer a more idiomatic approach.

-      const responseArray = [];
-      for await (const item of response) {
-        responseArray.push(item);
-      }
+      const responseArray = await Array.fromAsync(response);

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d268b9a and 0b0b043.

⛔ Files ignored due to path filters (1)

• pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (6)

• components/barcode_lookup/actions/batch-barcode-lookup/batch-barcode-lookup.mjs (1 hunks)
• components/barcode_lookup/actions/get-product-by-barcode/get-product-by-barcode.mjs (1 hunks)
• components/barcode_lookup/actions/search-products-by-parameters/search-products-by-parameters.mjs (1 hunks)
• components/barcode_lookup/barcode_lookup.app.mjs (1 hunks)
• components/barcode_lookup/common/utils.mjs (1 hunks)
• components/barcode_lookup/package.json (2 hunks)

🧰 Additional context used

🧬 Code graph analysis (4)

components/barcode_lookup/actions/get-product-by-barcode/get-product-by-barcode.mjs (2)

components/barcode_lookup/actions/batch-barcode-lookup/batch-barcode-lookup.mjs (1)

• response (24-29)

components/barcode_lookup/actions/search-products-by-parameters/search-products-by-parameters.mjs (1)

• response (79-95)

components/barcode_lookup/actions/search-products-by-parameters/search-products-by-parameters.mjs (2)

components/barcode_lookup/actions/batch-barcode-lookup/batch-barcode-lookup.mjs (1)

• response (24-29)

components/barcode_lookup/actions/get-product-by-barcode/get-product-by-barcode.mjs (1)

• response (23-28)

components/barcode_lookup/actions/batch-barcode-lookup/batch-barcode-lookup.mjs (3)

components/barcode_lookup/actions/get-product-by-barcode/get-product-by-barcode.mjs (1)

• response (23-28)

components/barcode_lookup/actions/search-products-by-parameters/search-products-by-parameters.mjs (1)

• response (79-95)

components/barcode_lookup/common/utils.mjs (2)

• parseObject (1-24)
• parseObject (1-24)

components/barcode_lookup/barcode_lookup.app.mjs (1)

components/spotify/actions/get-album-tracks/get-album-tracks.mjs (1)

• axios (53-56)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)

• GitHub Check: pnpm publish
• GitHub Check: Publish TypeScript components
• GitHub Check: Lint Code Base
• GitHub Check: Verify TypeScript components

🔇 Additional comments (1)

components/barcode_lookup/actions/search-products-by-parameters/search-products-by-parameters.mjs (1)

77-103: LGTM: Pagination and success handling are well-implemented.

The use of app.paginate with the searchProducts function is appropriate, and the success summary correctly reports the number of products retrieved.

[michelle0927]

LGTM!