Skip to content

feat: Rent-a-Relic Market - Book Authenticated Vintage Compute (#2312)#1812

Open
wsimon1982 wants to merge 1 commit intoScottcjn:mainfrom
wsimon1982:feat/issue-2312-rent-a-relic
Open

feat: Rent-a-Relic Market - Book Authenticated Vintage Compute (#2312)#1812
wsimon1982 wants to merge 1 commit intoScottcjn:mainfrom
wsimon1982:feat/issue-2312-rent-a-relic

Conversation

@wsimon1982
Copy link
Contributor

Closes Scottcjn/rustchain-bounties#2312

Implementation

This PR adds the core backend logic for the Rent-a-Relic Market allowing AI agents to book time on authenticated vintage machines.

  • FastAPI Core: GET /api/relics, POST /api/reserve, POST /api/pay, POST /api/execute
  • wRTC Escrow verification hooks (Simulated deposit lock mechanisms)
  • Provenance Receipts: Ed25519 signatures over the execution hash to cryptographically verify that the payload ran on the target relic.

Tested locally. Payout wallet (RTC): RTC1274aea37cc74eb889bf2abfd22fee274fc37706b

@wsimon1982 wsimon1982 requested a review from Scottcjn as a code owner March 23, 2026 14:30
@github-actions github-actions bot added documentation Improvements or additions to documentation BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) ci size/XL PR: 500+ lines labels Mar 23, 2026
@wsimon1982
Copy link
Contributor Author

Hey @Scottcjn,

I noticed that the BCOS v2 Engine Scan in this PR failed, but this is a workflow permission issue rather than a code error. The scan itself actually passed successfully (Trust Score: 60/100 | Tier L1 met: True), but the GitHub Action threw an HttpError: Resource not accessible by integration (403) when it tried to post the results back as a comment on this PR.

This happens because the workflow doesn't have the required issues=write and pull_requests=write permissions to post comments from a forked PR.

Adding the following to the BCOS v2 workflow YAML should fix it for future external contributions:

permissions:
  pull-requests: write
  issues: write

The relic_marketplace.py code itself works flawlessly. Let me know if everything is good to merge for the 150 RTC bounty! 🚀

@Scottcjn
Copy link
Owner

Good submission for bounty #2312 — the FastAPI marketplace code with Ed25519 provenance receipts is solid. A few things to fix before merge:

Must fix:

  1. Remove whitespace-only changes in 4 unrelated files: .github/workflows/labeler.yml, .github/workflows/stale.yml, site/beacon/vehicles.js, and wallet/coinbase_wallet.py. These are CRLF-to-LF line ending conversions that touch files you did not intend to modify. Please configure git config core.autocrlf false and drop those files from the PR.

Nice to have:
2. session_id uses MD5 — consider secrets.token_hex(16) instead (MD5 has collisions)
3. The time.sleep(1) in /api/execute would block the event loop in production

Once the whitespace noise is removed, this is mergeable. You were first to submit and the code is functional. Please push a fix and I will merge.

Wallet noted: RTC1274aea37cc74eb889bf2abfd22fee274fc37706b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) ci documentation Improvements or additions to documentation size/XL PR: 500+ lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rent-a-Relic Market — Book Authenticated Vintage Compute (150 RTC)

2 participants