chore: rm orphan notion-sync.ts copies#32
Conversation
Both copies were MD5-identical (45723e4b30dfd91864ec2d7cd6954abc) and outside tsconfig.json include (client/src, shared, server) — never built, never imported. Refs: chittycanon://gov/killlist/2026-06-02-notion Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
💤 Files with no reviewable changes (2)
📝 WalkthroughWalkthroughThis PR removes the entire Notion integration module, deleting 553 lines of code. The NotionSyncManager class and setupNotionIntegration function are removed, eliminating functionality for creating Notion session dashboards, managing task cards, syncing state from Neon databases, and reflecting GitHub Actions status in Notion callouts. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a340eb6ddc
ℹ️ 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".
| @@ -1,553 +0,0 @@ | |||
| import { Client } from '@notionhq/client'; | |||
There was a problem hiding this comment.
Restore Notion init script target
When NOTION_TOKEN is set, both cross-session-sync/init-sync.sh:122-126 and src/cross-session-sync/init-sync.sh:122-126 enter notion-integration and run npx tsx notion-sync.ts create-dashboard; this deletion removes the only tracked file in that directory, so a fresh checkout no longer has the directory or script and initialization aborts under Notion-configured environments. Either keep this target or update the init scripts to use the replacement implementation before deleting it.
Useful? React with 👍 / 👎.
Part of notion search-and-destroy. Both copies MD5-identical and outside tsconfig include — unbuilt, unimported. See chittycanon://gov/killlist/2026-06-02-notion.
Summary by CodeRabbit