Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
33 changes: 33 additions & 0 deletions .github/workflows/publish-typescript-sdk.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,33 @@
name: Publish TypeScript SDK

on:
release:
types: [published]

jobs:
publish:
if: startsWith(github.event.release.tag_name, 'sdks/typescript/')
Copy link

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

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

The publish job filter checks for tags starting with sdks/typescript/, but release-please tags are typically generated with a hyphenated prefix (e.g., sdks/typescript-v0.2.0) and/or the configured component (e.g., typescript-sdk-v0.2.0). As written, this condition is likely to never match, so the SDK won’t publish on release. Update the prefix to match the actual tag format produced by your release-please configuration (or adjust the release-please component/tag settings to emit the expected prefix).

Suggested change
if: startsWith(github.event.release.tag_name, 'sdks/typescript/')
if: startsWith(github.event.release.tag_name, 'sdks/typescript-v')

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Check this ☝️

runs-on: ubuntu-latest
permissions:
contents: read
id-token: write
defaults:
run:
working-directory: sdks/typescript
steps:
- name: ⬇️ Checkout repo
uses: actions/checkout@v6

- name: 😻 Setup Node.js
uses: actions/setup-node@v6
with:
node-version: 22
registry-url: https://registry.npmjs.org

- name: 📥 Install dependencies
uses: bahmutov/npm-install@v1
with:
working-directory: sdks/typescript

- name: 📦 Publish to npm
run: npm publish --provenance --access public
5 changes: 3 additions & 2 deletions .release-please-manifest.json
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
{
".": "1.0.0"
}
"sdks/typescript": "0.2.0",
"evals/prompts": "1.2.0"
Copy link

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

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

The manifest pins evals/prompts at 1.2.0, but evals/prompts/CHANGELOG.md already contains a 1.3.0 entry (and the TS SDK changelog references prompts v1.3.0). If 1.3.0 is the current released state, release-please will compute the next prompts release/tag incorrectly. Align the manifest version with the latest released prompts version (or revert/update the prompts changelog if 1.3.0 hasn’t been released).

Suggested change
"evals/prompts": "1.2.0"
"evals/prompts": "1.3.0"

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Check this ☝️

}
51 changes: 27 additions & 24 deletions release-please-config.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
{
"release-type": "simple",
"include-v-in-tag": true,
"pull-request-title-pattern": "chore${scope}: release${component} ${version}",
"changelog-sections": [
Expand All @@ -13,48 +12,52 @@
"section": "Bug Fixes",
"hidden": false
},
{
"type": "chore",
"section": "Miscellaneous",
"hidden": false
},
{
"type": "docs",
"section": "Documentation",
"hidden": false
},
{
"type": "perf",
"section": "Performance Improvements",
"hidden": false
},
{
"type": "test",
"section": "Tests",
"type": "revert",
"section": "Reverts",
"hidden": false
},
{
"type": "refactor",
"section": "Code Refactoring",
"type": "docs",
"section": "Documentation",
"hidden": false
},
{
"type": "deps",
"type": "build",
Copy link

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

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

In the changelog section mapping, commit type build is labeled as the "Dependencies" section. This is confusing because dependency updates are commonly tagged deps/chore(deps) (and build usually reads as build-system changes). Consider either renaming the section to reflect build changes, or reintroducing a dedicated deps section and mapping it to "Dependencies".

Suggested change
"type": "build",
"type": "build",
"section": "Build System",
"hidden": false
},
{
"type": "deps",

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Check this ☝️

"section": "Dependencies",
"hidden": false
},
{
"type": "security",
"section": "Security",
"hidden": false
"type": "chore",
"section": "Miscellaneous",
"hidden": true
},
{
"type": "revert",
"section": "Reverts",
"hidden": false
"type": "test",
"section": "Tests",
"hidden": true
},
{
"type": "refactor",
"section": "Code Refactoring",
"hidden": true
}
],
"packages": {
".": {}
"sdks/typescript": {
"release-type": "node",
"changelog-path": "CHANGELOG.md",
"component": "typescript-sdk"
},
"evals/prompts": {
"release-type": "simple",
"changelog-path": "CHANGELOG.md",
"component": "prompts"
}
}
}
}
2 changes: 2 additions & 0 deletions sdks/typescript/README.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
# @learning-commons/evaluators

[![npm version](https://img.shields.io/npm/v/@learning-commons/evaluators)](https://www.npmjs.com/package/@learning-commons/evaluators)

TypeScript SDK for Learning Commons educational text complexity evaluators.

## Installation
Expand Down
2 changes: 1 addition & 1 deletion sdks/typescript/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@learning-commons/evaluators",
"version": "0.1.0",
"version": "0.2.0",
"description": "TypeScript SDK for Learning Commons educational evaluators",
"type": "module",
"types": "./dist/index.d.ts",
Expand Down
Loading