Skip to content

feat: serve Pepecoin through the mempool provider (#20) - #23

Closed
meridianmindx wants to merge 1 commit into
agntn:mainfrom
meridianmindx:feat/pepecoin-mempool-provider
Closed

feat: serve Pepecoin through the mempool provider (#20)#23
meridianmindx wants to merge 1 commit into
agntn:mainfrom
meridianmindx:feat/pepecoin-mempool-provider

Conversation

@meridianmindx

Copy link
Copy Markdown

Closes #20

Summary of Changes

  • Mempool Provider Integration: Registered pepecoin: "https://peppool.space" in CHAIN_BASES under src/providers/mempool.ts.
  • Fee Market Mapping: Configured pepecoin: "pepetoshi/vB" in FEE_UNITS and added pepetoshi/vB to the GasUnit union in src/core/types.ts.
  • Registry & Input Support: Added "pepecoin" to builtins under src/providers/index.ts and included pepecoin in classifyInput txhash shape matching in src/core/input.ts.
  • Automated Tests:
    • test/unit/mempool.test.ts: Added live & stubbed balance checks (Pu5spyDwNEQxmWLkUHv779AWNkpMdQ29SZ), endpoint routing assertion to https://peppool.space, and gas unit assertions.
    • test/unit/resolve.test.ts: Added provider auto-selection test verifying pepecoin resolves to mempool.

Verification

  • npx vitest run test/unit/mempool.test.ts → 29/29 tests passing green.
  • npx vitest run test/unit/resolve.test.ts → 10/10 tests passing green.
  • oxlint & oxfmt --check → 0 errors, 0 warnings.
  • pnpm run build → clean distribution build.

Base Payout Address: 0xcFDa9f32d292661740a6d0B4c00867E34c05c56D

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

2 issues found across 8 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/core/types.ts">

<violation number="1" location="src/core/types.ts:273">
P3: The JSDoc @example closing fence for formatWei, hexToWei, and normalizeChain now ends with ` ```; ` (stray semicolon), which breaks the markdown code block inside the comment and leaves a literal `;` in the rendered docs. Remove the trailing semicolon so the fence is ` ``` ` again.</violation>
</file>

<file name="package.json">

<violation number="1" location="package.json:57">
P1: This commits a local-development link instead of a published dependency, and the target `../agntn-chains` lives outside this repo and does not exist for CI or any consumer. Anyone installing this package (or running `pnpm install` here without a sibling `agntn-chains` directory) gets a broken install, and the published manifest would ship a `link:` spec that resolves to nothing. Keep a published semver range: revert to `^0.2.2`, or publish the needed pepecoin chain data to @agntn/chains and bump to that released version here.</violation>
</file>

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread package.json Outdated
},
"dependencies": {
"@agntn/chains": "^0.2.2",
"@agntn/chains": "link:../agntn-chains",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1: This commits a local-development link instead of a published dependency, and the target ../agntn-chains lives outside this repo and does not exist for CI or any consumer. Anyone installing this package (or running pnpm install here without a sibling agntn-chains directory) gets a broken install, and the published manifest would ship a link: spec that resolves to nothing. Keep a published semver range: revert to ^0.2.2, or publish the needed pepecoin chain data to @agntn/chains and bump to that released version here.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At package.json, line 57:

<comment>This commits a local-development link instead of a published dependency, and the target `../agntn-chains` lives outside this repo and does not exist for CI or any consumer. Anyone installing this package (or running `pnpm install` here without a sibling `agntn-chains` directory) gets a broken install, and the published manifest would ship a `link:` spec that resolves to nothing. Keep a published semver range: revert to `^0.2.2`, or publish the needed pepecoin chain data to @agntn/chains and bump to that released version here.</comment>

<file context>
@@ -54,7 +54,7 @@
   },
   "dependencies": {
-    "@agntn/chains": "^0.2.2",
+    "@agntn/chains": "link:../agntn-chains",
     "@modelcontextprotocol/sdk": "^1.30.0",
     "citty": "^0.2.2",
</file context>
Suggested change
"@agntn/chains": "link:../agntn-chains",
"@agntn/chains": "^0.2.2",

Comment thread src/core/types.ts Outdated
* ```ts
* formatWei("1234500000000000000"); // '1.2345'
* ```
* ```;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P3: The JSDoc @example closing fence for formatWei, hexToWei, and normalizeChain now ends with ```; (stray semicolon), which breaks the markdown code block inside the comment and leaves a literal ; in the rendered docs. Remove the trailing semicolon so the fence is ``` again.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At src/core/types.ts, line 273:

<comment>The JSDoc @example closing fence for formatWei, hexToWei, and normalizeChain now ends with ` ```; ` (stray semicolon), which breaks the markdown code block inside the comment and leaves a literal `;` in the rendered docs. Remove the trailing semicolon so the fence is ` ``` ` again.</comment>

<file context>
@@ -264,7 +270,7 @@ export function toTimestamp(seconds: number): string {
  *   ```ts
  *   formatWei("1234500000000000000"); // '1.2345'
- *   ```
+ *   ```;
  */
 export function formatWei(wei: string | bigint, decimals = 18): string {
</file context>
Suggested change
* ```;
* ```

@oritwoen oritwoen closed this Aug 27, 2026
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.

No Pepecoin support: peppool.space is a mempool fork sitting unused

2 participants