Fix build/compile breakage across example projects (DOC-1995) - #46
Merged
Elijah Skeirik (payablijah) merged 2 commits intoAug 10, 2026
Merged
Conversation
- sdk/go-sdk: stop gitignoring go.sum so `go build` works on a fresh
clone instead of failing with missing go.sum entries
- webhooks/php-sdk: update stale composer.lock to satisfy the
payabli/payabli ^1.0.8 constraint (was locked to 1.0.1, which also
pulled in a vulnerable guzzlehttp/guzzle transitively)
- webhooks/ts-sdk: fix eventType casing ("ApprovedPayment" ->
"approvedpayment") to match the current @payabli/sdk-node types
- vue-integration: fix package.json name copy-pasted from
react-integration
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
react-native was pinned at 0.86.0, two patches behind what Expo SDK
57.0.12 expects (0.86.2), which caused react-native-webview's WebView
component to fail tsc's overload resolution ("props: never"). Bumping
react-native-webview alone didn't fix it; aligning react-native via
`expo install --fix` did. Confirmed `expo export --platform web` still
bundles correctly.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Copilot started reviewing on behalf of
Elijah Skeirik (payablijah)
August 10, 2026 17:37
View session
There was a problem hiding this comment.
Pull request overview
Fixes build and type-check failures across several SDK and framework examples.
Changes:
- Adds Go dependency checksums and tracks
go.sum. - Refreshes PHP dependencies and corrects the TypeScript webhook event literal.
- Aligns React Native with Expo and corrects Vue package metadata.
Reviewed changes
Copilot reviewed 4 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
sdk/go-sdk/.gitignore |
Allows go.sum tracking. |
sdk/go-sdk/go.sum |
Adds dependency checksums. |
webhooks/php-sdk/composer.lock |
Updates incompatible and vulnerable dependencies. |
webhooks/ts-sdk/main.ts |
Uses the SDK-compatible event literal. |
react-native/package.json |
Aligns React Native with Expo SDK 57. |
vue-integration/package.json |
Corrects the Vue package name. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.
Summary
Maintenance pass over every example project in the repo (DOC-1995) — installed/built/compiled all 21 SDK, webhook, and framework-integration examples to find real gaps, not just style nits. Fixed the ones that actually break for a fresh clone or fail type-checking:
sdk/go-sdk:go buildfailed on a fresh clone withmissing go.sum entryerrors becausesdk/go-sdk/.gitignoreexplicitly excludedgo.sum. Removed that line and committed a workinggo.sum.webhooks/php-sdk:composer installfailed outright (exit 4) —composer.lockwas pinned topayabli/payabli1.0.1, which doesn't satisfy the^1.0.8constraint incomposer.json. Rancomposer updateto fix; this also cleared 11 security advisories on a transitively-pinned oldguzzlehttp/guzzle.webhooks/ts-sdk:tscfailed —eventType: "ApprovedPayment"no longer matches the current@payabli/sdk-nodetypes, which expect the lowercase literal"approvedpayment". Fixed the casing.react-native:tsc --noEmitfailed onPayabliEmbeddedWebView.tsxwith a WebView prop-type overload error. Root cause wasreact-nativeitself being two patch versions behind what the installed Expo SDK (57.0.12) expects (0.86.0vs0.86.2), notreact-native-webview. Bumped viaexpo install --fix; confirmedexpo export --platform webstill bundles correctly.vue-integration:package.jsonnamefield wasreact-embedded-demo, copy-pasted fromreact-integration. Fixed tovue-embedded-demo.Every other example (boarding, temp-token, react-integration, sdk/ts-sdk, sdk/cs-sdk, webhooks/cs-sdk, sdk & webhooks go/java/php/py/ruby/rust) installed and built/compiled cleanly with no changes needed.
Test plan
sdk/go-sdk:go build ./...succeeds on a clean checkoutwebhooks/php-sdk:composer installsucceeds,composer auditreports no advisorieswebhooks/ts-sdk:tsc --noEmitpassesreact-native:tsc --noEmitpasses,expo export --platform webbundles successfullyvue-integration:pnpm buildstill succeeds after the name change🤖 Generated with Claude Code