Skip to content

feat(satp-hermes): add adapter layer#4097

Merged
RafaelAPB merged 1 commit intohyperledger-cacti:mainfrom
RafaelAPB:feat-4017/api3-interface-rebase-with-upstream
Feb 26, 2026
Merged

feat(satp-hermes): add adapter layer#4097
RafaelAPB merged 1 commit intohyperledger-cacti:mainfrom
RafaelAPB:feat-4017/api3-interface-rebase-with-upstream

Conversation

@RafaelAPB
Copy link
Copy Markdown
Contributor

@RafaelAPB RafaelAPB commented Jan 22, 2026

Pull Request Requirements

  • Rebased onto upstream/main branch and squashed into single commit to help maintainers review it more efficient and to avoid spaghetti git commit graphs that obfuscate which commit did exactly what change, when and, why.
  • Have git sign off at the end of commit message to avoid being marked red. You can add -s flag when using git commit command. You may refer to this link for more information.
  • Follow the Commit Linting specification. You may refer to this link for more information.

Character Limit

  • Pull Request Title and Commit Subject must not exceed 72 characters (including spaces and special characters).
  • Commit Message per line must not exceed 80 characters (including spaces and special characters).

A Must Read for Beginners
For rebasing and squashing, here's a must read guide for beginners.

Summary:
SATP Hermes adapter layer, API3 updates

Features:
-Add SATP Hermes adapter layer with webhook flows, configuration validation, and new adapter-focused tests.
-Expand SATP Hermes docs and diagrams, update bundled OpenAPI specs, and refresh example configs.
-Centralize database configuration and apply package/test updates across the repo.

@RafaelAPB RafaelAPB changed the title Feat 4017/api3 interface rebase with upstream feat«(satp-hermes): add adapter layer Jan 22, 2026
@RafaelAPB RafaelAPB changed the title feat«(satp-hermes): add adapter layer feat:(satp-hermes): add adapter layer Jan 22, 2026
@RafaelAPB RafaelAPB force-pushed the feat-4017/api3-interface-rebase-with-upstream branch from f3bd8d0 to 6d241d0 Compare January 22, 2026 18:30
@RafaelAPB RafaelAPB changed the title feat:(satp-hermes): add adapter layer feat(satp-hermes): add adapter layer Jan 22, 2026
@RafaelAPB RafaelAPB force-pushed the feat-4017/api3-interface-rebase-with-upstream branch from 6d241d0 to 4b4d2b0 Compare January 22, 2026 18:36
@RafaelAPB RafaelAPB force-pushed the feat-4017/api3-interface-rebase-with-upstream branch 4 times, most recently from df89206 to bfa22a8 Compare January 23, 2026 18:09
@RafaelAPB RafaelAPB force-pushed the feat-4017/api3-interface-rebase-with-upstream branch from bfa22a8 to 8743e3b Compare January 30, 2026 16:58
Copy link
Copy Markdown
Contributor

@LordKubaya LordKubaya left a comment

Choose a reason for hiding this comment

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

Please write the PR description.

RafaelAPB

This comment was marked as duplicate.

@RafaelAPB RafaelAPB force-pushed the feat-4017/api3-interface-rebase-with-upstream branch from 91b3287 to 52a9c12 Compare February 3, 2026 18:35
Copy link
Copy Markdown
Contributor

@LordKubaya LordKubaya left a comment

Choose a reason for hiding this comment

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

LGTM

@RafaelAPB RafaelAPB force-pushed the feat-4017/api3-interface-rebase-with-upstream branch 3 times, most recently from 5d761d3 to 6259e00 Compare February 4, 2026 21:23
@RafaelAPB RafaelAPB enabled auto-merge (rebase) February 4, 2026 21:25
@RafaelAPB RafaelAPB force-pushed the feat-4017/api3-interface-rebase-with-upstream branch from 6259e00 to c6a1d7e Compare February 6, 2026 16:26
Copy link
Copy Markdown
Member

@AndreAugusto11 AndreAugusto11 left a comment

Choose a reason for hiding this comment

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

Thanks @RafaelAPB, amazing additions!!

@LordKubaya
Copy link
Copy Markdown
Contributor

Please rebase to merge this branch.

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>
Co-authored-by: Rafael Belchior <rafael.belchior@omnumi.io>

ci(satp-hermes): update satp hermes

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>

fix: enhance code robustness

cactus common fix: expect escapeHtml to be string
add checks to satp adapter test server

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>

fix: fix examples build

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>

fix(satp-hermes): centralize database config

Adds enhanced timeout config to knex by:

- Create db-config.ts
- Add WAL mode and busy_timeout to SQLite
configs to prevent SQLITE_BUSY errors
- Configure connection pool settings
 (min/max, acquire/create/destroy timeouts)
- Update knexfile.ts and knexfile-remote.ts
 to use centralized config
- Add createEnhancedTimeoutConfig to Docker
 tests for PostgreSQL databases
- Re-export database config utilities from
test-utils.ts

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>

fix(satp-hermes): fix package.json

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>

fix: update dev container

CI is failing due to expired GPG keys for Yarn
The devcontainer feature is trying to install Yarn via APT
but the repository signature is invalid.
Using Corepack instead

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>

chore: add satp reports to ignore

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>

test(satp-hermes): fix tests

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>

chore(satp-hermes): improve debugging

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>

fix(satp-hermes): incorporate feedback

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>

chore(satp-hermes): update diagram generation

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>

ci: improve cleanup

Signed-off-by: Rafael Belchior <rafael.belchior@tecnico.ulisboa.pt>
@RafaelAPB RafaelAPB force-pushed the feat-4017/api3-interface-rebase-with-upstream branch from c6a1d7e to e3abf6e Compare February 26, 2026 00:04
@RafaelAPB RafaelAPB merged commit 8a940fd into hyperledger-cacti:main Feb 26, 2026
124 of 144 checks passed
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.

4 participants