Skip to content

Feature/boe c2 fixes #29

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
manishdex25 wants to merge 15 commits into
base: beta
Choose a base branch
from
Open

Feature/boe c2 fixes #29

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
f7c0c78
feat: add obligation escrow commands and types
manishdex25 Jul 27, 2026
99ffec9
docs: update README for improved clarity and formatting
manishdex25 Jul 28, 2026
0c0bfed
fix: update W3C obligation record with new issuer and restructure cre…
manishdex25 Jul 28, 2026
9802e14
refactor: consolidate verification commands into a unified pipeline
manishdex25 Jul 28, 2026
1e5f18a
docs: enhance README with updated TrustVC library details and local d…
manishdex25 Jul 30, 2026
a28a14e
chore(release): add beta-boe branch for prerelease channel
manishdex25 Jul 30, 2026
81a4d24
chore(release): update to version 1.1.0 and enhance dependencies
manishdex25 Jul 31, 2026
c431c3a
docs: update README for obligation registry usage and enhance error h…
manishdex25 Jul 31, 2026
fef3701
chore(release): update release configuration and CI workflows
manishdex25 Jul 31, 2026
5488f9c
docs: refine README formatting and clarify command usage
manishdex25 Jul 31, 2026
dedcad6
chore: update dependencies and improve error handling in obligation e…
manishdex25 Jul 31, 2026
3390fcc
Update .github/workflows/ci.yml
manishdex25 Jul 31, 2026
4c3f853
Update src/utils/obligation-document.ts
manishdex25 Jul 31, 2026
ffe1544
Refactor error handling in extractObligationDocumentInfo function
manishdex25 Jul 31, 2026
6dad862
Update README and command descriptions for obligation-escrowd
manishdex25 Jul 31, 2026
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
9 changes: 8 additions & 1 deletion .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,14 +1,21 @@
name: CI

# Same checks for PRs targeting main or beta (and for pushes to those branches).
on:
push:
branches:
- main
- beta
pull_request:
branches:
- main
- beta

jobs:
test:
name: Test
runs-on: ubuntu-latest

strategy:
matrix:
node-version: [22.x, 24.x]
Expand Down
11 changes: 11 additions & 0 deletions .github/workflows/release.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,13 @@
name: Release

# Publishes to npm on push to:
# - main → stable (@latest)
# - beta → prerelease (@beta), e.g. 1.2.0-beta.1
on:
push:
branches:
- main
- beta

jobs:
test:
Expand Down Expand Up @@ -39,6 +43,11 @@ jobs:
name: Release
needs: test
runs-on: ubuntu-latest
permissions:
contents: write
issues: write
pull-requests: write
id-token: write

steps:
- name: Checkout code
Expand All @@ -52,6 +61,7 @@ jobs:
with:
node-version: 22.x
cache: 'npm'
registry-url: 'https://registry.npmjs.org/'

@coderabbitai coderabbitai Bot Jul 31, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🩺 Stability & Availability | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

if rg -n 'registry-url|NODE_AUTH_TOKEN' .github/workflows/release.yml; then
  echo "Remove setup-node npm authentication for semantic-release." >&2
  exit 1
fi

Repository: TrustVC/trustvc-cli

Length of output: 328

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "## release workflow around setup-node"
sed -n '50,85p' .github/workflows/release.yml || true

echo
echo "## workflow trigger and dependencies"
sed -n '1,45p' .github/workflows/release.yml || true

echo
echo "## package type and scripts"
python3 - <<'PY'
import json
from pathlib import Path
p=Path('package.json')
if p.exists():
    data=json.loads(p.read_text())
    for k in ['name','type','scripts','devDependencies','dependencies']:
        if k in data:
            print(f"{k}:", data[k])
PY

Repository: TrustVC/trustvc-cli

Length of output: 2992

Remove setup-node npm authentication for semantic-release.

setup-node creates an .npmrc when registry-url is set, and semantic-release npm authentication can conflict with that file. Use NPM_TOKEN for token authentication, or switch fully to npm Trusted Publishing, but do not configure both mechanisms.

Proposed fix 
       with:
         node-version: 22.x
         cache: 'npm'
-        registry-url: 'https://registry.npmjs.org/'

         env:
           GITHUB_TOKEN: ${{ secrets.GH_TOKEN }}
           NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
-          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.github/workflows/release.yml at line 64, Remove the registry-url
configuration from the setup-node step in the semantic-release workflow so it no
longer creates an .npmrc; retain a single authentication mechanism by using
NPM_TOKEN or fully adopting npm Trusted Publishing, without configuring both.


- name: Install dependencies
run: npm ci
Expand All @@ -64,3 +74,4 @@ jobs:
env:
GITHUB_TOKEN: ${{ secrets.GH_TOKEN }}
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
16 changes: 14 additions & 2 deletions .releaserc.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
{
"branches": ["main"],
"branches": [
"main",
{
"name": "beta",
"prerelease": "beta",
"channel": "beta"
}
],
"plugins": [
"@semantic-release/commit-analyzer",
"@semantic-release/release-notes-generator",
Expand All @@ -9,7 +16,12 @@
"changelogFile": "CHANGELOG.md"
}
],
"@semantic-release/npm",
[
"@semantic-release/npm",
{
"npmPublish": true
}
],
[
"@semantic-release/git",
{
Expand Down
768 changes: 572 additions & 196 deletions README.md
View file
Open in desktop

Large diffs are not rendered by default.

Loading
Loading