Skip to content

[WC-xxxx] chore: Remove LAST Custom Widget#2316

Merged
samuelreichert merged 3 commits into
mainfrom
chore/remove-custom-widgets
Jul 9, 2026
Merged

[WC-xxxx] chore: Remove LAST Custom Widget#2316
samuelreichert merged 3 commits into
mainfrom
chore/remove-custom-widgets

Conversation

@samuelreichert

Copy link
Copy Markdown
Contributor

Pull request type

Dependency changes (any modification to dependencies in package.json)
Refactoring (e.g. file rename, variable rename, etc.)


Description

Remove customWidgets folder

The last custom widget (calendar-custom-web) has been removed, so this cleans up the now-empty packages/customWidgets/ folder and all remaining references to it.

Changes

  • Remove calendar-custom-web widget and the customWidgets/ folder
  • Drop the obsolete react-big-calendar@0.19.2 patch (only used by the deleted widget; calendar-web uses ^1.19.4 and is unaffected)
  • Remove customWidgets references from copilot-instructions.md, Renovate ignorePaths, and the update-screenshots CATEGORY_DIRS

No functional impact — workspace globs (packages//) self-adjust and pnpm-lock.yaml was regenerated.

samuelreichert and others added 3 commits July 9, 2026 11:19
Last custom widget removed; folder no longer exists. Workspace globs
(packages/*/*) self-adjust and pnpm-lock.yaml regenerated.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Patch only applied to the deleted calendar-custom-web (v0.19.2).
calendar-web uses react-big-calendar ^1.19.4 and is unaffected.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Remove customWidgets from copilot repo context, renovate ignorePaths,
and update-screenshots CATEGORY_DIRS now that the folder is gone.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@samuelreichert samuelreichert requested a review from a team as a code owner July 9, 2026 09:24
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

AI Code Review

⚠️ Approved with suggestions — low-severity items only, safe to merge


What was reviewed

File Change
.github/copilot-instructions.md Removed packages/customWidgets/* from monorepo description
.github/renovate.json Removed ignorePaths: ["**/customWidgets/**"] rule
automation/scripts/update-screenshots.mjs Removed "customWidgets" from CATEGORY_DIRS set
package.json Removed react-big-calendar@0.19.2 patch entry from pnpm.patchedDependencies
packages/customWidgets/calendar-custom-web/** Entire package deleted (all source, config, tests, CHANGELOG)
patches/react-big-calendar@0.19.2.patch Patch file deleted (only used by the deleted widget)
pnpm-lock.yaml Regenerated; orphaned transitive deps removed

Skipped (out of scope): pnpm-lock.yaml (lockfile, no independent review needed beyond pairing with package.json — confirmed clean)


Findings

⚠️ Low — Placeholder JIRA ticket number in PR title

File: PR title
Note: The title contains [WC-xxxx] which looks like an unfilled placeholder. Either replace it with a real ticket number, or drop the JIRA prefix and rely on the conventional-commit chore: prefix alone (both formats are accepted per repo conventions).


⚠️ Low — Snyk security check shows 3 failures

File: CI checks — security/snyk (AppDev AppStore content)
Note: The check reports "3 tests have failed". Since this PR only deletes code, it cannot introduce new vulnerabilities — these failures are very likely pre-existing across other packages. Confirm with the team that these Snyk findings are known/pre-existing and not newly surfaced by the dependency graph change (removing react-big-calendar@0.19.2 and its transitive deps could theoretically surface a previously-suppressed finding). No action required if pre-existing.


Positives

  • Complete cleanup confirmed: a repo-wide grep finds zero remaining customWidgets references after the change.
  • The react-big-calendar@0.19.2 patch and all its orphaned transitive dependencies (dnd-core, redux@3.7.2/@4.2.1, react-dnd, react-overlays@0.7.4, warning@2.x/3.x, symbol-observable, etc.) are cleanly removed from the lockfile — good dependency hygiene.
  • Removing the ignorePaths from Renovate means the remaining packages will now properly receive automated dependency updates — a correct and forward-looking cleanup.
  • The CATEGORY_DIRS set in the screenshot automation script is trimmed precisely, with no over-removal.

@samuelreichert samuelreichert merged commit d0f87d3 into main Jul 9, 2026
13 of 15 checks passed
@samuelreichert samuelreichert deleted the chore/remove-custom-widgets branch July 9, 2026 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants