Add H-E-B platform support and fix CONNECTORS_PATH#72
Open
Conversation
- Add HEB to platform registry (icon, color, display name) - Add PlatformHebIcon component with official app icon - Fix sync-connectors-dev.js to use CONNECTORS_PATH env var as the source directory, so `CONNECTORS_PATH=../data-connectors npm run tauri:dev` actually syncs from a local data-connectors checkout Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add heb-playwright entry to connectors/registry.json - Sync script now auto-detects connector dirs by checking for *-playwright.json metadata files instead of a hardcoded skip list - Filter .git paths during cpSync to avoid copying nested git dirs Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The sync script now reads connector directories directly from registry.json file paths — the single source of truth — instead of scanning the filesystem with naming convention heuristics. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
1 participant
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
PlatformHebIconcomponent using the official app icon (512x512 PNG)sync-connectors-dev.jsto actually useCONNECTORS_PATHenv var as the connector source directoryCONNECTORS_PATH fix
Previously, setting
CONNECTORS_PATHonly skipped the remote fetch infetch-connectors.jsbut didn't affectsync-connectors-dev.js, which always read from the project's ownconnectors/directory. Now:Actually syncs connectors from your local data-connectors checkout into
~/.dataconnect/connectors/.Test plan
CONNECTORS_PATH=../data-connectors npm run tauri:devsyncs connector files correctly🤖 Generated with Claude Code