Skip to content

Nequi Web Sandbox#7

Merged
pulgueta merged 4 commits intomainfrom
pulgueta/web-sandbox
Jan 29, 2026
Merged

Nequi Web Sandbox#7
pulgueta merged 4 commits intomainfrom
pulgueta/web-sandbox

Conversation

@pulgueta
Copy link
Owner

No description provided.

@changeset-bot
Copy link

changeset-bot bot commented Jan 28, 2026

⚠️ No Changeset found

Latest commit: d9e7d67

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@pulgueta
Copy link
Owner Author

pulgueta commented Jan 28, 2026

hey @greptile plz review

@greptile-apps
Copy link

greptile-apps bot commented Jan 28, 2026

Greptile Overview

Greptile Summary

This PR adds a comprehensive web-based sandbox application for testing the Nequi Node.js SDK. The sandbox provides an interactive UI for all SDK features including QR payments, push payments, subscriptions, dispersions, and reports.

Key Changes:

  • Created Astro-based sandbox application in docs/ directory with SSR support
  • Implemented dual-mode credential handling: environment variables for local dev, session storage for production
  • Added API routes wrapper using Elysia to expose all SDK methods via HTTP
  • Created comprehensive UI pages for testing each SDK feature with forms and result displays
  • Updated root README with improved documentation and examples
  • Bumped package version from 1.0.0 to 1.6.0
  • Updated release config to use correct changelog path

Architecture:

  • The sandbox uses Astro with Node.js adapter for server-side rendering
  • Credentials are validated using Zod schemas for type safety
  • In production mode, credentials are stored in browser sessionStorage (client-side only)
  • In development mode, credentials are loaded from .env.local file
  • All API calls follow consistent error handling pattern with [error, data] tuple responses

The implementation is well-structured with proper separation of concerns between configuration, SDK instantiation, API routes, and UI pages.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The PR adds a well-structured sandbox application with proper error handling, input validation using Zod, and clear separation of concerns. The code follows consistent patterns, uses TypeScript for type safety, and implements sensible security practices (credentials in sessionStorage, not persisted on server). No breaking changes to the main SDK package.
  • No files require special attention

Important Files Changed

Filename Overview
docs/src/pages/api/[...slug].ts API routes for Nequi SDK operations with proper error handling
docs/src/lib/config.ts Environment configuration with Zod validation for credentials
docs/src/lib/nequi.ts SDK instance factory with dual environment support
docs/src/layouts/Layout.astro Main layout with credential management modal and navbar
README.md Updated with comprehensive documentation and examples
packages/server/package.json Version bump from 1.0.0 to 1.6.0

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@pulgueta pulgueta merged commit 179ae77 into main Jan 29, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant