Skip to content

feat(nodes): add Dev.to integration node#43

Closed
adityasai1234 wants to merge 7 commits into
wespreadjam:mainfrom
adityasai1234:feat/devto-node
Closed

feat(nodes): add Dev.to integration node#43
adityasai1234 wants to merge 7 commits into
wespreadjam:mainfrom
adityasai1234:feat/devto-node

Conversation

@adityasai1234
Copy link
Copy Markdown
Contributor

Closes #34

Changes

  • Built a new devtoNode under integrations/devto/devto.ts
  • Fully typed using Zod with implementations for:
    • createArticle
    • updateArticle
    • getArticles
  • Validated via [test.ts](cci:7://file:///Users/medow/Documents/jam-nodes/test.ts:0:0-0:0)

Node Credentials

Requires context.credentials.devto.apiKey to authenticate POST/PUT operations.

…ion in fetchWithRetry

- seo-audit: both branches of the POSITIVE_CHECKS if/else were
  assigning `checkData.passed` identically, making the distinction
  a no-op. Normal checks now correctly invert with `!checkData.passed`
  so that a DataForSEO "passed=true" (problem absent) maps to isPassed=true.

- fetchWithRetry: a 429 on the final retry attempt fell through the
  if-block and returned the rate-limit Response to the caller instead
  of throwing. Now throws FetchRetryError("Rate limit exceeded after N
  attempts") consistent with how 5xx exhaustion is handled.
- Add utils/resolve-path.ts as canonical path resolver
- Supports dot notation, keyed array (contacts[0]), standalone
  index ([0].name), and empty paths
- map.ts now imports from shared utility
- Align cpc in SeoKeywordResearchOutputSchema to match DataForSeoKeyword
- Add `sideEffects: false` to both core and nodes packages for tree-shaking
- Clean up root README and copy to packages for npm publish visibility
- Define Dev.to API key in NodeCredentials
- Implement createArticle, updateArticle, and getArticles operations
- Export devtoNode through integration barrels
- Add devtoNode to builtInNodes and verify in test suite
@MAlshaik
Copy link
Copy Markdown
Contributor

MAlshaik commented Mar 3, 2026

Closing as duplicate of #44 which was merged with a more comprehensive implementation (27 tests vs 3 tests). Thank you for your contribution!

@MAlshaik MAlshaik closed this Mar 3, 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.

[Integration] Dev.to - Developer blogging

2 participants