Skip to content

Comments

OPS-1526: Minor updates README and core input.#15

Merged
apujari-aembit merged 3 commits intoOPS-1485from
OPS-1526
Dec 29, 2025
Merged

OPS-1526: Minor updates README and core input.#15
apujari-aembit merged 3 commits intoOPS-1485from
OPS-1526

Conversation

@apujari-aembit
Copy link
Collaborator

@apujari-aembit apujari-aembit commented Dec 29, 2025

Related Issue / Ticket

Ref: https://aembit.atlassian.net/browse/OPS-1526

Security Considerations

Checklist

  • I have commented my code, particularly in hard-to-understand areas.

  • Documentation: I have made corresponding changes to the documentation (README.md, action.yml, etc.).

  • Security: I have verified to the best of my ability that this change introduces no new security vulnerabilities.

  • Build: I have run $ yarn build before my final commit to have an up-to-date distribution of the Action.

@apujari-aembit apujari-aembit changed the title Ops 1526 OPS-1526: Minor updates README and core input. Dec 29, 2025
@apujari-aembit apujari-aembit merged commit 424a0fb into OPS-1485 Dec 29, 2025
2 checks passed
@apujari-aembit apujari-aembit deleted the OPS-1526 branch December 29, 2025 18:27
thedevelopnik added a commit that referenced this pull request Dec 29, 2025
* refactor: reorganize OpenAPI spec location and code generation output

- Move edge-api.yaml from __test__/resources to resources/
- Update kubb config to output generated code to gen/ instead of __test__/gen/
- Adjust tsconfig.json rootDir
- Update vitest coverage exclusions for new gen/ directory

* add kubb client plugin for API client generation

- Add @kubb/plugin-client dependency
- Configure client generation in kubb.config.ts with fetch client
- Rename npm script from test:generate to generate
- Generate new API client code from OpenAPI spec

* feat: add support for multiple credential types

Add support for:
- UsernamePassword
- OAuthToken
- GoogleWorkloadIdentityFederation
- AwsStsFederation

Changes:
- Create new credential.ts module with getCredential and setOutputs functions
- Refactor main.ts to use new credential module instead of api-key module
- Update validate.ts to use generated credential type enums
- Add validateServerPort function with proper number validation

* test: update tests for multi-credential type support

- Add comprehensive credential.test.ts for new credential module
- Update main.test.ts to mock credential module instead of api-key
- Update all test imports to use new generated types location
- Add server port validation tests
- Update test mocks to handle multiple credential types
- Remove old api-key.test.ts as it's been replaced by credential.test.ts

* documents all supported credential types and outputs

* excludes generated code from linting and formatting

* updates build command and build distribution

* adds explicit content-type header to edge api calls

* add tests for content-type header being sent to edge api

* adds requirements and troubleshooting info to README

* OPS-1526: Minor updates README and core input. (#15)

* (feat): GitHub markdowns to emphasize critical information

* (feat): flag to trim whitespace.

* removes alpha release warning

---------

Co-authored-by: apujari-aembit <apujari@aembit.io>
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.

2 participants