Skip to content

feat(vendor): add The Graph + pattern: Reproducible Audit Extraction (supersedes #186) - #193

Merged
ethsystems-bot merged 4 commits into
masterfrom
centaur/the-graph-review-fixes
Sep 9, 2026
Merged

feat(vendor): add The Graph + pattern: Reproducible Audit Extraction (supersedes #186)#193
ethsystems-bot merged 4 commits into
masterfrom
centaur/the-graph-review-fixes

Conversation

@ethsystems-bot

Copy link
Copy Markdown
Contributor

Lands @p-diogo's work from #186, with the review comments applied.

#186 could not be updated in place. Its head branch lives on graphprotocol/map, an org-owned fork. GitHub reports maintainer_can_modify: true on that PR, and the flag does not hold for org-owned forks. Maintainers get push: false. This branch carries Pedro's original commit cde2eb9 unmodified so the content is his, then adds the review fixes on top.

Authored by @p-diogo. Closes #186.

Review comments applied

Five inline comments from @rymnc on #186.

  • pattern-reproducible-audit-extraction.md L10: applied the suggested wording
  • pattern-reproducible-audit-extraction.md L63: answers "do all of these implement reproducible audit extraction?"
  • the-graph.md L11: applied the suggested title
  • the-graph.md L15: applied the suggested wording
  • the-graph.md L41: hosted-delivery bullet made concise, and its "but ... but" construction removed

On the implementations list

Not all six implemented the pattern. Each entry now names the leg it covers.

  • Firehose: extraction
  • Substreams: transform
  • graph-node: transform, with proofs of indexing
  • TrueBlocks: extraction plus manifest, publishing IPFS-addressed chunks whose publisher pointers are recorded on-chain
  • cryo: extraction alone, carrying no manifest or attestation

Subsquid is dropped. It publishes no determinism, reproducibility, content-addressing or proof-of-indexing guarantee. It is an ETL toolkit rather than an instance of this pattern.

Vale

Both files are wholly new, so reviewdog treats every line as changed. pattern-reproducible-audit-extraction.md:77 and the-graph.md:76 each contained only, which trips EthSystems.Marketing. Both are cleared.

Prompted by: ac

p-diogo and others added 4 commits August 6, 2026 00:34
Applies the five inline comments from @rymnc on #186.

- rae L10: applied the suggested wording
- rae L63: answers "do all of these implement reproducible audit extraction?".
  Each entry now names the leg it covers. Subsquid is dropped, since it
  publishes no determinism, reproducibility, content-addressing or
  proof-of-indexing guarantee
- the-graph L11: applied the suggested title
- the-graph L15: applied the suggested wording
- the-graph L41: hosted-delivery bullet made concise

Also clears two EthSystems.Marketing hits on "only". Both files are wholly new,
so Vale sees every line as changed and the check would fail.
@ethsystems-bot
ethsystems-bot merged commit 6cb1527 into master Sep 9, 2026
4 checks passed
@ethsystems-bot
ethsystems-bot deleted the centaur/the-graph-review-fixes branch September 9, 2026 10:42
rymnc pushed a commit to ethsystems/web that referenced this pull request Sep 11, 2026
Bumps the `content` submodule (ethsystems/map) from
[`1221c36`](ethsystems/map@1221c36)
to
[`b468053`](ethsystems/map@b468053)
— 10 commits, 9 new pages.

| Type | Added | PR |
| --- | --- | --- |
| Vendor | Bermuda | [#191](ethsystems/map#191)
|
| Vendor | Inco | [#177](ethsystems/map#177) |
| Vendor | Interfold |
[#194](ethsystems/map#194) |
| Vendor | The Graph |
[#193](ethsystems/map#193) |
| Pattern | Confidential Policy Verdicts |
[#183](ethsystems/map#183) |
| Pattern | Ephemeral Committees |
[#194](ethsystems/map#194) |
| Pattern | Private Geospatial Attestation |
[#184](ethsystems/map#184) |
| Pattern | Reproducible Audit Extraction |
[#193](ethsystems/map#193) |
| Pattern | Verifiable DKG & Threshold Decryption |
[#194](ethsystems/map#194) |

Also included: Privacy Pools deployment status/link correction
([#192](ethsystems/map#192)), 18 terminology
fixes across existing pages
([#188](ethsystems/map#188)), and Vale CI config
fixes ([#195](ethsystems/map#195),
[#196](ethsystems/map#196)).

Prompted by: ac

Co-authored-by: ethsystems-bot <321619048+ethsystems-bot@users.noreply.github.com>
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.

2 participants