Repository navigation
D-API-0: map the documented API surface onto deliverables - #40
Merged
Merged
Conversation
data/hubspot/surface_map.tsv assigns each of the 140 spec families to the D-id that owns it, or UNPLANNED with a reason. hubspo-harvest deliverables counts functions per D-id (current, legacy-only, beta-only) and refuses a family with no row. A test pins the committed map to the committed surface and to STATUS_BOARD: every family mapped, no stale row, every D-id real, 1,518 functions in total. Largest owners: D-API-1 object CRUD 545, D-MKT-4 CMS 285, D-COM-1 commerce 70, D-MKT-2 marketing email 59. 134 functions in 11 groups have no deliverable: ISS-SURFACE-UNPLANNED. D-ONT-6 and D-API-0 marked shipped (#38, #39). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PPcakjk7ioQsnasmNDtc5z
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PPcakjk7ioQsnasmNDtc5z
AdaWorldAPI
marked this pull request as ready for review
October 21, 2026 00:03
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 893b6449ca
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
A beta endpoint whose reference page also exists in legacy, with no current counterpart, was counted as legacy-only and again as beta-only. 12 such pages (six workflow, six form pages) inflated D-FLOW-1 and D-MKT-1 by six each and the total to 1,518. Beta-only now excludes pages legacy already has: 1,506 distinct functions, 86 beta-only. The unit test gains the twin case; board figures corrected where they cited 1,518 or 98 beta-only. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PPcakjk7ioQsnasmNDtc5z
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PPcakjk7ioQsnasmNDtc5z
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.
What
data/hubspot/surface_map.tsvassigns each of the 140 spec families of HubSpot's documented API to the D-id that owns it, or toUNPLANNEDwith a reason.surface::familystrips the directory,.jsonand the version suffix, sov3/crm-contacts-v3.jsonandcrm-contacts-v2026-09.jsonare bothcrm-contacts.hubspo-harvest deliverables <surface.tsv> <map.tsv>counts functions per D-id: current, legacy-only and beta-only. A family with no row is an error, never dropped.STATUS_BOARD;Result (measured 2026-10-20)
The full table is in
LATEST_STATE.Unplanned (
ISS-SURFACE-UNPLANNED, operator decision): 134 functions in 11 groups.The board also marks D-ONT-6 and D-API-0 shipped (#38, #39).
Verification
cargo fmt --check,clippy -D warningsandcargo test --workspaceall pass.Cross-session claim: Linear EXO-50
session_01PPcakjk7ioQsnasmNDtc5z/005.Generated by Claude Code