Repository navigation
feat(payment): add payment-ppp-pos skill - #91
Merged
Merged
Conversation
Help Alliances partners scaffold VTEX Sales App POS connectors from the official PPF example instead of inventing an ecommerce card flow. Co-authored-by: Cursor <cursoragent@cursor.com>
🛡️ SDD Check — action requiredI couldn't detect an SDD in this PR. Please check one option below (requires write access to the repo):
|
Contributor
|
✅ Validation passed — all checks OK Full validation output |
…output Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Isabella Veloso Sá <77300325+Isabella-Veloso@users.noreply.github.com>
Use SDK Direct Sale helpers, align charge start with the POS guide, and make overlapping Create Payment writes concurrency-safe. Co-authored-by: Cursor <cursoragent@cursor.com>
The Claude exporter lowercases the skill description; keep the generated file in that form so CI export check stays green. Co-authored-by: Cursor <cursoragent@cursor.com>
Corrected capitalization and punctuation for consistency and clarity in the guidance text.
The Claude exporter lowercased entire skill descriptions, so restoring names like VTEX Sales App always failed the exports-up-to-date check. Lowercase only the first character so generated files keep source casing. Co-authored-by: Cursor <cursoragent@cursor.com>
brunoamui
approved these changes
Sep 23, 2026
brunoamui
left a comment
Collaborator
There was a problem hiding this comment.
Analisei a formatação e a aderência à pasta tracks. Tudo certo
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
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.
Description
Add a Payment track skill for Payment Provider Protocol on VTEX Sales App (POS), covering card-present methods that the existing ecommerce connector skills do not.
Venda Direta Credito/Venda Direta Debito, two-phase Payment Apps, null Gateway card data, and IO retry vs standalone notificationContribution Checklist
tracks/{track}/skills/{name}/skill.md, not inskills/,exports/, orrules/directoriesskill.md(lowercase), notSKILL.mdor other variantsname,description, andmetadatawithtrackandtagsfieldsbun run validateand all hard checks passbun run exportand committed both source and generated filesNote: local Windows
bun run validatefailstrack-consistencyon every skill (path.split(/)vs\). Ubuntu CI is the intended gate.Public Documentation Sync
Public docs update required — list the affected pages and link the follow-up below:
VTEX Skills → intro skill count 40 → 41; Payment track 5 → 6 in the Tracks and skills table
Follow-up: open a PR in
vtex/dev-portal-contentafter this merges (not filed yet)Test plan
bun run validatepasses on Ubuntuexports/,rules/,skills/up to date) passespayment-ppp-posappears in Cursor, Copilot, Claude, AGENTS.md, OpenCode, and Kiro exportsgeneration.md,ppf-scaffold.md,examples.md)Related Issues
N/A