Repository navigation
feat(server): escrow relay protocol models, migration and config (#96) - #123
Closed
moises-cisneros wants to merge 3 commits into
Closed
moises-cisneros wants to merge 3 commits into
moises-cisneros wants to merge 3 commits into
Conversation
Narrow rules scoped by file path and line shape for env var name constants, generated class-name lookups and recorded ledger keys. Migrates the existing allowlist to [[allowlists]], which cannot be mixed with the deprecated [allowlist].
Adds PreparedTransaction, CreateHireResult, HireDetail, Hire, Payment and Puls3ApiException, the escrow_preparation table and the requestId, input and jobId columns of hire. Part of #96.
Reads the preparation validity, inclusion fee and platform fee from the environment with no defaults (ADR-0005 D3). Documents the keys in .env.example. Part of #96.
This was referenced Oct 8, 2026
Contributor
Author
TOMOKI977
approved these changes
Oct 8, 2026
TOMOKI977
left a comment
Contributor
There was a problem hiding this comment.
Approved. This slice is part of the code I reviewed in full on #119 at b0a4adc (all four lenses; see the review there). I checked that the tip of the chain matches b0a4adc except for the CI fix in .env.example and .gitleaks.toml (and main's #112). The follow-ups listed on #119 apply to the chain as a whole.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Refs #96
Part of the chain that replaces #119, split after its review. Tracker: #122.
Summary
Protocol models (
PreparedTransaction,CreateHireResult,HireDetail,Hire,Payment,Puls3ApiException), theescrow_preparationtable, the newhirecolumns, the relay configuration and the session wallet port. Also the CI fixes from the review of #119: the three new variables in.env.exampleand narrow gitleaks allowlists.Acceptance criteria
The chain as a whole covers #96; this PR carries its slice (see Scope below).
prepare…andsubmitEscrowCall(proven in the docs PR)submit…call never throws for chain outcomesChainSubmissionVerification evidence
Notes for reviewers
feat/96-escrow-relay), notmain.size:exception.Chain Context
feat/96-escrow-relayfeat/96-escrow-relayHireRelayConfig,SessionWallet,.env.example,.gitleaks.toml.Chain Overview
Scope
HireRelayConfig,SessionWallet,.env.example,.gitleaks.toml.Autonomy