feature/meshstack 2025.53 #1
Merged
Conversation
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
Member
JohannesRudolph
commented
Dec 10, 2025
JohannesRudolph
commented
- Update action to use node24
- chore: standardize build configuration and CI workflow
- Align README: naming conventions and structure with related actions
- feat: separate auth from register-source, use token file from well-known location
- test: add token file reading tests and fix dist rebuild
- feat: support building block run URL input with fallback to payload
- refactor: break down run() function into smaller, focused functions
- refactor: extract input extraction and register source logic into functions
- refactor: make actions/core mockable
- Refactor GitHub context handling in registerBuildingBlockSource
- docs: update readme
- Update package.json to specify Node.js 24 runtime - Standardize build scripts using @vercel/ncc - Add Node.js native test framework support - Update tsconfig.json to ES2020 target with consistent compiler options - Add ci.yml workflow with Node 24, tests, and build verification - Remove unnecessary Jest dependencies
- Add buildingBlockRunUrl input to action.yml for fetching building block run data - Update index.ts to handle both URL-based and payload-based inputs - URL-based input makes a GET request to fetch the building block run object - Maintains backward compatibility with existing buildingBlockRun payload input - Extract common logic for parsing building block run data from both sources - Add tests for building block run parsing from URL and payload sources
- Extract loadTokenFromFile() to handle token file reading and validation - Extract loadBuildingBlockRunFromBase64() to handle base64 decoding of payload input - Extract loadBuildingBlockRunFromUrl() to handle HTTP GET requests for URL input - Add TypeScript interfaces for TokenData and BuildingBlockRun for better type safety - Simplify run() to orchestrate the helper functions and register the source - Improves code readability and testability by separating concerns
…ctions - Extract extractInputs() to handle input parsing and GitHub output setting - Extract registerSource() to handle the POST request to meshStack API - Extract buildRequestPayload() to construct the request payload - Extract buildRequestHeaders() to construct the request headers - Add TypeScript interfaces for better type safety: - BuildingBlockInput: represents individual building block inputs - RequestSource: represents the source section of the request payload - RequestPayload: represents the complete request payload - RequestHeaders: represents all HTTP headers with proper index signature - ExtractedInputs: type-safe dictionary for extracted inputs - Simplify run() to orchestrate the helper functions - All tests pass
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.