Skip to content

Conversation

michelle0927
Copy link
Collaborator

@michelle0927 michelle0927 commented Sep 18, 2025

Updates notion_api_key app to import changes from #18212 and #18349

Summary by CodeRabbit

  • Chores
    • Bumped Notion API Key package version to 0.1.3.
    • Updated Notion SDK dependency to ^1.0.2.
    • Aligned version metadata to 0.0.3 across all Notion actions and sources, including: Append Block, Create Comment, Create Page, Create Page from Database, Duplicate Page, Query Database, Retrieve Block, Retrieve Database Content, Retrieve Database Schema, Retrieve Page, Retrieve Page Property Item, Search, Update Page, and sources for New Comment Created, New Database, New Page, Page or Subpage Updated, Updated Page ID, and Updated Page.

Copy link

vercel bot commented Sep 18, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
pipedream-docs Ignored Ignored Sep 18, 2025 8:00pm
pipedream-docs-redirect-do-not-edit Ignored Ignored Sep 18, 2025 8:00pm

@michelle0927 michelle0927 self-assigned this Sep 18, 2025
Copy link
Contributor

coderabbitai bot commented Sep 18, 2025

Walkthrough

Updates the Notion API Key component: increments version metadata from 0.0.2 to 0.0.3 across multiple actions and sources, and bumps the component package version to 0.1.3 with a dependency update for @pipedream/notion from ^0.10.1 to ^1.0.2. No behavioral or control-flow changes.

Changes

Cohort / File(s) Summary
Actions: version bump
components/notion_api_key/actions/.../append-block.mjs, .../create-comment.mjs, .../create-page-from-database.mjs, .../create-page.mjs, .../duplicate-page.mjs, .../query-database.mjs, .../retrieve-block.mjs, .../retrieve-database-content.mjs, .../retrieve-database-schema.mjs, .../retrieve-page-property-item.mjs, .../retrieve-page.mjs, .../search.mjs, .../update-page.mjs
Update exported version from "0.0.2" to "0.0.3"; no logic changes.
Sources: version bump
components/notion_api_key/sources/.../new-comment-created.mjs, .../new-database.mjs, .../new-page.mjs, .../page-or-subpage-updated.mjs, .../updated-page-id.mjs, .../updated-page.mjs
Update exported version from "0.0.2" to "0.0.3"; no logic changes.
Package metadata
components/notion_api_key/package.json
Bump package version 0.1.2 → 0.1.3; update dependency @pipedream/notion ^0.10.1 → ^1.0.2.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Suggested labels

User submitted

Suggested reviewers

  • andrewjschuang

Poem

A hop and a bump, then another small leap,
Version nibble to nibble, our garden we keep.
Pages and blocks in a tidy new key,
Depend-a-dandelions swaying with glee.
Thump-thump! says the changelog—concise and spry,
Carrots for builds, and we’re ready to fly. 🥕🐇

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The PR description indicates the high-level intent (importing changes referenced by two issue links) but does not follow the repository's required template header "## WHY" and is missing details about the rationale, the exact dependency/version bump, which files were changed, and any testing or compatibility notes, so it is incomplete relative to the template. The current text is helpful but too brief for reviewers to fully assess impact. Please update the PR description to use the repository template by adding a "## WHY" section that explains why @pipedream/notion is being bumped, summarizes the concrete changes (package.json dependency bump and per-file version updates), links the referenced issue numbers, and notes any testing, migration, or compatibility considerations so reviewers can assess impact.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title "Notion API Key - update @pipedream/notion version" is concise, specific, and directly reflects the primary change in the changeset (updating the @pipedream/notion dependency and associated version bumps in the notion_api_key component), so it communicates the main intent clearly to reviewers. It avoids noise and is easy to scan in the project history.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch notion-api-key-update

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6a0fa50 and 35bf1fd.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (20)
  • components/notion_api_key/actions/append-block/append-block.mjs (1 hunks)
  • components/notion_api_key/actions/create-comment/create-comment.mjs (1 hunks)
  • components/notion_api_key/actions/create-page-from-database/create-page-from-database.mjs (1 hunks)
  • components/notion_api_key/actions/create-page/create-page.mjs (1 hunks)
  • components/notion_api_key/actions/duplicate-page/duplicate-page.mjs (1 hunks)
  • components/notion_api_key/actions/query-database/query-database.mjs (1 hunks)
  • components/notion_api_key/actions/retrieve-block/retrieve-block.mjs (1 hunks)
  • components/notion_api_key/actions/retrieve-database-content/retrieve-database-content.mjs (1 hunks)
  • components/notion_api_key/actions/retrieve-database-schema/retrieve-database-schema.mjs (1 hunks)
  • components/notion_api_key/actions/retrieve-page-property-item/retrieve-page-property-item.mjs (1 hunks)
  • components/notion_api_key/actions/retrieve-page/retrieve-page.mjs (1 hunks)
  • components/notion_api_key/actions/search/search.mjs (1 hunks)
  • components/notion_api_key/actions/update-page/update-page.mjs (1 hunks)
  • components/notion_api_key/package.json (2 hunks)
  • components/notion_api_key/sources/new-comment-created/new-comment-created.mjs (1 hunks)
  • components/notion_api_key/sources/new-database/new-database.mjs (1 hunks)
  • components/notion_api_key/sources/new-page/new-page.mjs (1 hunks)
  • components/notion_api_key/sources/page-or-subpage-updated/page-or-subpage-updated.mjs (1 hunks)
  • components/notion_api_key/sources/updated-page-id/updated-page-id.mjs (1 hunks)
  • components/notion_api_key/sources/updated-page/updated-page.mjs (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
  • GitHub Check: pnpm publish
  • GitHub Check: Publish TypeScript components
  • GitHub Check: Lint Code Base
  • GitHub Check: Verify TypeScript components
🔇 Additional comments (20)
components/notion_api_key/actions/search/search.mjs (1)

14-14: LGTM: version bump only.

Consistent with the package upgrade; no behavioral changes.

components/notion_api_key/sources/updated-page/updated-page.mjs (1)

14-14: LGTM: version bump only.

Matches the pattern used across sources; no functional deltas.

components/notion_api_key/actions/update-page/update-page.mjs (1)

14-14: LGTM: version bump only.

No changes beyond metadata; safe to merge.

components/notion_api_key/actions/retrieve-database-schema/retrieve-database-schema.mjs (1)

14-14: LGTM: version bump only.

No runtime impact observed.

components/notion_api_key/actions/retrieve-page/retrieve-page.mjs (1)

14-14: LGTM: version bump only.

In line with other action modules.

components/notion_api_key/actions/query-database/query-database.mjs (1)

14-14: LGTM: version bump only.

No changes to props or execution.

components/notion_api_key/sources/page-or-subpage-updated/page-or-subpage-updated.mjs (1)

14-14: LGTM: version bump only.

Aligned with other source bumps; no code-path changes.

components/notion_api_key/actions/append-block/append-block.mjs (1)

14-14: Verify Notion component versions & imports

Automated check returned no matches — run the script locally with the repository checked out and confirm:

  • components/notion_api_key/{actions,sources}: every component.version == "0.0.3"
  • components/notion_api_key/package.json: package version matches component versions and "dependencies"."@pipedream/notion" is in sync
  • All imports under components/notion_api_key reference @pipedream/notion paths consistent with the v1 actions/sources layout
components/notion_api_key/actions/create-comment/create-comment.mjs (1)

14-14: Version bump looks fine; confirm upstream action path compatibility with @pipedream/notion v1.x.

No logic change. Please verify the import path still exists in @pipedream/notion@^1.0.2.

components/notion_api_key/actions/retrieve-page-property-item/retrieve-page-property-item.mjs (1)

14-14: Version metadata updated; double-check upstream export and event/prop shape.

Ensure the v1.x package didn’t rename or relocate actions/retrieve-page-property-item/retrieve-page-property-item.mjs.

components/notion_api_key/sources/updated-page-id/updated-page-id.mjs (1)

14-14: Source version bump ok; validate source schema stability under @pipedream/notion v1.x.

Confirm emitted item structure and dedupe ID logic remain unchanged.

components/notion_api_key/actions/retrieve-database-content/retrieve-database-content.mjs (1)

14-14: LGTM on version bump; confirm upstream path and params unchanged.

Sanity-check retrieve-database-content input/return types didn’t shift in v1.x.

components/notion_api_key/actions/create-page/create-page.mjs (1)

14-14: Version bump acknowledged; ensure upstream action signature unchanged.

Verify create-page required props and response shape match v1.x.

components/notion_api_key/actions/duplicate-page/duplicate-page.mjs (1)

14-14: OK to bump; confirm duplicator action path still resolved in v1.x.

A quick import resolution check across the repo is recommended.

components/notion_api_key/actions/retrieve-block/retrieve-block.mjs (1)

14-14: LGTM on metadata; validate retrieve-block behavior parity.

Spot-check for any renamed props/outputs in the upstream v1.x action.

components/notion_api_key/package.json (1)

3-3: Major dep bump to @pipedream/notion v1.x — verify Node/ESM compatibility and import paths

  • Web checks: @notionhq/client v4.0.2 requires Node >=18; @pipedream/notion appears to preserve actions/* and sources/* import paths.
  • Local repo inspection failed here (node_modules missing). Locally, after installing deps, confirm:
    • project runtime / engines >= Node 18;
    • whether @pipedream/notion v1.x is ESM-only (check package.json "type") and adjust imports/build if so;
    • all "@pipedream/notion/..." imports in components/** still resolve.

Run locally after installing deps:
npm ci || pnpm install
rg -nP "from ['"]@pipedream/notion(/[^'\"])?['"]" -g "components/**.mjs" -S || true
fd . node_modules/@pipedream/notion -t f | sed 's#.
/@pipedream/notion/##' | sort -u | head -n 200
jq -r '.type? // empty' node_modules/@pipedream/notion/package.json || true
jq -r '.engines? // empty' node_modules/@notionhq/client/package.json || true

components/notion_api_key/sources/new-database/new-database.mjs (1)

14-14: LGTM: version bump only

Consistent with the other source wrappers; no runtime impact.

components/notion_api_key/sources/new-comment-created/new-comment-created.mjs (1)

14-14: LGTM: version bump only

Matches the pattern used across sources/actions; safe to merge.

components/notion_api_key/sources/new-page/new-page.mjs (1)

14-14: LGTM — version bump verified

Overrides common.version as intended; no behavior change. All components under components/notion_api_key are at 0.0.3 (no 0.0.2 found) and components/notion_api_key/package.json lists "@pipedream/notion": "^1.0.2".

components/notion_api_key/actions/create-page-from-database/create-page-from-database.mjs (1)

14-14: Approve: version bump — @pipedream/notion v1.0.2 import paths unchanged

Confirmed @pipedream/notion v1.0.2 still exposes @pipedream/notion/actions/create-page-from-database/create-page-from-database.mjs and @pipedream/notion/sources/new-page/new-page.mjs; no import changes required.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@GTFalcao GTFalcao merged commit 373c2cf into master Sep 18, 2025
10 checks passed
@GTFalcao GTFalcao deleted the notion-api-key-update branch September 18, 2025 20:17
@github-project-automation github-project-automation bot moved this from Ready for PR Review to Done in Component (Source and Action) Backlog Sep 18, 2025
sergio-eliot-rodriguez pushed a commit to sergio-eliot-rodriguez/sergio_wong_does_pipedream that referenced this pull request Sep 21, 2025
* update @pipedream/notion dependency version

* pnpm-lock.yaml
vunguyenhung added a commit that referenced this pull request Sep 24, 2025
* Leonardo AI components

* added unzoom image action

* fixing link errors

* more lint fixes

* Merging pull request #18359

* fix: pagination prop and params struct

* fix: no need for paginate here

* chore: update version

* chore: cleanup

* chore: update package

* feat: allow raw response

* chore: bump package

* fix: buffer response instead

* Update components/google_drive/actions/download-file/download-file.mjs

Co-authored-by: Jorge Cortes <jacortesmahmud@gmail.com>

* versions

* pnpm-lock.yaml

* pnpm-lock.yaml

* pnpm-lock.yaml

* feat: add content selector

* chore: bump package

* fix: comments

* chore: bump versions

* chore: fix versions

* fixes: QA fixes

* feat: add cursor to req

* package.json

---------

Co-authored-by: joao <joao@coform.com>
Co-authored-by: joaocoform <joao@usecoform.com>
Co-authored-by: Jorge Cortes <jacortesmahmud@gmail.com>
Co-authored-by: Michelle Bergeron <michelle.bergeron@gmail.com>
Co-authored-by: Luan Cazarine <luanhc@gmail.com>

* Merging pull request #18361

* update siteId prop

* pnpm-lock.yaml

* package.json version

* Google Sheets - update row refresh fields  (#18369)

* change prop order and refresh fields

* bump package.json

* Pipedrive - fix app name (#18370)

* use pipedriveApp instead of app

* bump package.json

* Pipedrive - pipelineId integer (#18372)

* pipelineId - integer

* bump versions

* Adding app scaffolding for lightspeed_ecom_c_series

* Adding app scaffolding for financial_data

* Adding app scaffolding for microsoft_authenticator

* Merging pull request #18345

* updates

* versions

* versions

* Merging pull request #18368

* updates

* remove console.log

* versions

* Coinbase Developer Platform - New Wallet Event (#18342)

* new component

* pnpm-lock.yaml

* updates

* updates

* Hubspot - update search-crm (#18360)

* update search-crm

* limit results to one page

* update version

* package.json version

* Merging pull request #18347

* widget props

* fix version

* Adding app scaffolding for rundeck

* Merging pull request #18378

* Update Taiga component with new actions and sources

- Bump version to 0.1.0 in package.json and add dependency on @pipedream/platform.
- Introduce new actions for creating, updating, and deleting issues, tasks, and user stories.
- Add sources for tracking changes and deletions of issues and tasks.
- Implement utility functions for parsing and cleaning objects in common/utils.mjs.
- Enhance prop definitions for better integration with Taiga API.

* pnpm update

* Refactor Taiga actions to utilize parseObject utility

- Added parseObject utility for tags, watchers, and points in update-issue, update-task, and update-userstory actions.
- Removed the update-project action as it is no longer needed.
- Enhanced base source to include secret key validation for webhook security.

* Merging pull request #18382

* add testSources prop

* pnpm-lock.yaml

* fix

* Merging pull request #18323

* Added actions

* Added actions

* Added actions

* Merging pull request #18377

* Added actions

* Update components/weaviate/actions/create-class/create-class.mjs

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

---------

Co-authored-by: Luan Cazarine <luanhc@gmail.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

* Merging pull request #18376

* Adding app scaffolding for etrusted

* Adding app scaffolding for intelliflo_office

* Adding app scaffolding for thoughtspot

* Adding app scaffolding for kordiam

* Adding app scaffolding for ticketsauce

* trustpilot fixes (#18152)

* trustpilot fixes

* more fixes

* update versions

* more version updates

* fixes

* Bump all Trustpilot actions to version 0.1.0

Major improvements and API updates across all actions:
- Enhanced private API support with proper authentication
- Improved parameter handling and validation
- Better error handling and response structures
- Added new conversation flow for product reviews
- Fixed endpoint URLs to match latest API documentation
- Streamlined request/response processing

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* up version and clean up sources

* merge

* fix business ID

* delete temp action

* Update components/trustpilot/sources/new-product-reviews/new-product-reviews.mjs

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

* Update components/trustpilot/sources/new-product-reviews/new-product-reviews.mjs

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

* Update components/trustpilot/sources/new-product-reviews/new-product-reviews.mjs

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

* Update components/trustpilot/sources/new-service-reviews/new-service-reviews.mjs

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

* comments

* Pagination

* fixes

* comments

* missed some `$`'s

* unduplicated

* more fixes

* final comments

* more comments

* .

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

* Adding app scaffolding for peekalink

* 18314 twilio (#18350)

* Update Twilio component versions and dependencies

- Update Twilio Send Message action adding detailed description for 'from' prop and refactoring phone number validation logic.
- Incremented action versions for several Twilio actions.

* pnpm update

* Updating LinkedIn API version (#18399)

* Merging pull request #18394

* Databricks API - Jobs action components (#18371)

* Notion property building improvements (#18381)

* validate property types

* versions

* Google Business - add debug log (#18407)

* add debug log

* bump versions

* Notion API Key - update @pipedream/notion version (#18409)

* update @pipedream/notion dependency version

* pnpm-lock.yaml

* Adding app scaffolding for reduct_video

* Adding app scaffolding for shopware

* Adding app scaffolding for instamojo

* Hubspot - bug fix to sources w/ property changes (#18379)

* updates

* versions

* Google sheets type fix (#18411)

* Fixing worksheetId prop type from string to integer

* Version bumps

---------

Co-authored-by: Leo Vu <vunguyenhung@outlook.com>

* Merging pull request #18393

* new components

* remove console.log

* versions

* update

* Merging pull request #18408

* 403 error message

* versions

* update

* Merging pull request #18419

* Changes per PR Review

* Removes leonardo_ai_actions.mdc not indented for merging

* synced lockfile after install

* fully lock form-data for leonardo_ai

* conflict solving

* lint fixes

* Chipped down Readme, implemented async options in gen motion

---------

Co-authored-by: jocarino <45713006+jocarino@users.noreply.github.com>
Co-authored-by: joao <joao@coform.com>
Co-authored-by: joaocoform <joao@usecoform.com>
Co-authored-by: Jorge Cortes <jacortesmahmud@gmail.com>
Co-authored-by: Michelle Bergeron <michelle.bergeron@gmail.com>
Co-authored-by: Luan Cazarine <luanhc@gmail.com>
Co-authored-by: michelle0927 <michelle0927@users.noreply.github.com>
Co-authored-by: Andrew Chuang <andrewjschuang@gmail.com>
Co-authored-by: danhsiung <35384182+danhsiung@users.noreply.github.com>
Co-authored-by: Lucas Caresia <lucascarezia@gmail.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: Job <9075380+Afstkla@users.noreply.github.com>
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Guilherme Falcão <48412907+GTFalcao@users.noreply.github.com>
Co-authored-by: Leo Vu <vunguyenhung@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants