Open
Conversation
The HEB connector manifest references icons/heb.svg but the file was missing. Added the official H-E-B red pill-shaped logo as a clean SVG vector. Also resolved merge conflict in registry.json to include both Oura and HEB/Whole Foods connectors. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Adds the full HEB connector: Playwright script for extracting profile, order history, and nutrition data from heb.com, with USDA FDC fallback for nutrition lookup. Includes schemas and official logo SVG. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replaced the generated wordmark SVG with the official H-E-B app icon (512x512 PNG) from the Apple App Store — the red rounded square with white H that matches the style of other connector icons. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Added heb/README.md documenting scopes, USDA_API_KEY env var, and nutrition lookup pipeline. Updated main README connector table to include HEB, Oura, Shop, and Instagram ads scope. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
volod-vana
approved these changes
Mar 12, 2026
Member
volod-vana
left a comment
volod-vana
left a comment
There was a problem hiding this comment.
Overall looks solid, few things:
- Version mismatch - metadata says 1.0.0 but registry says 1.1.0
- page.httpFetch used for USDA calls - dont think thats in our page API? If playwright-runner doesnt support it this will crash. All other connectors use page.evaluate + fetch()
- Schema heb.nutrition.json is missing blocked field in coverage properties but the connector emits it
Minor:
- company field is "HEB" uppercase, every other connector uses lowercase ("openai", "github" etc)
- Step 2 progress still shows step: 1 instead of step: 2
- IIFE return value on last line is unused, runner uses page.setData('result', ...) not the return
Rest looks good - bot detection + CAPTCHA recovery is well done, USDA fallback with match scoring is smart, proper page.evaluate('string') usage throughout.
Great job
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
heb/heb-playwright.js) — Playwright-based extraction of profile, order history, and product nutrition data from heb.comscoreMatch()validation and Foundation data type fallbackicons/heb.svg)heb.profile,heb.orders,heb.nutritionTest plan
icons/heb.svgrenders correctly🤖 Generated with Claude Code