Open
Conversation
…ype generation Adds 6 Data Access V1 contracts (ProtocolConfig, AttestationPolicy, DatasetRegistry, VanaRuntimeServers, VanaRuntimePermissions, AccessSettlement) to enable protocol configuration and runtime permission management. Automates SERVER_PATHS generation by extracting endpoint paths directly from OpenAPI spec using operationIds converted to camelCase, eliminating manual maintenance when server API changes.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Findings
|
…P methods Fixes dynamic import to use .ts extension instead of .js for contracts.config, preventing ERR_MODULE_NOT_FOUND errors. Iterates through all HTTP methods per path to avoid silently dropping endpoints when a path defines multiple operations.
- Add RuntimePermissionsController for dataset monetization via runtime - Add AccessSettlementController for operation payment settlement - Add DLP Operations page to vana-console for permission management - Add type-safe RuntimeGrantFile to relayer union types - Export new controllers and types from SDK entry points - Add utility functions for runtime grant file handling
…ions controllers - Add 16 tests for AccessSettlementController (read/write operations) - Add 15 tests for RuntimePermissionsController (CRUD operations) - Test error handling, wallet validation, and contract interactions - Coverage now at 76.18% (above 76% threshold) - All tests passing (2211 passed, 3 skipped)
- Replace placeholder permissionId (0n) with actual event parsing - Use waitForTransactionEvents to extract PermissionCreated event - Add test for missing event error case - Update test expectations to match real permission IDs 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Add inline comments to contracts.config.ts documenting which contracts should have their ABIs fetched from Moksha testnet: - Data Access V1 contracts (not yet deployed to Mainnet) - UniswapV3NonfungiblePositionManager (Mainnet contract not verified) - DataLiquidityPool (template contract with identical ABIs) These comments serve as the source of truth for ABI generation strategy until Data Access V1 deploys to Mainnet. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
tnunamak
changed the title
tnunamak
changed the title
Fix production readiness issues in dataset controller and console UI: SDK Changes: - Fix template literal type violations in DatasetController error handling - Add proper type guards for error messages (5 locations) Console Changes: - Fix Promise type annotation in file loading - Consolidate duplicate wagmi imports - Remove unused imports (Checkbox, Filter, UserFile) - Fix floating promises in useEffect hooks - Fix confusing void expressions in event handlers - Fix Select component items prop type error All validation checks now pass: lint, typecheck, and tests. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Remove requirement for NEXT_PUBLIC_PERSONAL_SERVER_BASE_URL environment variable in vana-console to fix Vercel deployment. Changes: - Default to empty string in SDKConfigProvider - Conditionally pass defaultPersonalServerUrl to SDK only when defined - Remove error throws in API routes for missing env var The personal server URL is optional and the SDK/app should work without it. Co-Authored-By: Claude <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.
Adds Data Access V1 contracts deployed on Moksha but not Mainnet:
New controllers:
accessSettlement,runtimePermissionsAlso automates SERVER_PATHS and schema type generation from OpenAPI spec.
ABI sources:
Public APIs exist for contracts not on Mainnet.