Skip to content

Extensions: Remove the Cron Manager extension - #1049

Merged
epeicher merged 1 commit into
trunkfrom
remove/cron-manager-extension
Oct 23, 2026
Merged

epeicher merged 1 commit into
trunkfrom
remove/cron-manager-extension

Conversation

@epeicher

@epeicher epeicher commented Oct 23, 2026 •

Copy link
Copy Markdown
Collaborator

What it does

Removes the bundled Cron Manager extension (extensions/desktop-mode-cron-manager/) and its PHPUnit test. The README now says the Cron Manager lives in desktop-mode-official-extensions.

Nothing in the core plugin loads it, and CI never built it. Its only tie to this repo was tests/phpunit/tests/cronManagerCapabilityGate.php.

Stacked on #1048. Merge that one first. This PR's diff is only the Cron Manager commit.

Rationale

Same reason as the Code Editor: it's an optional sibling plugin that already lives in the official-extensions repository, so keeping a second copy here means two copies drifting apart.

The copy in the official-extensions repo is brought up to date, including the multisite manage_network gate, in AllTerrainDeveloper/desktop-mode-official-extensions#4.

Implementation

  • Deletes extensions/desktop-mode-cron-manager/ and tests/phpunit/tests/cronManagerCapabilityGate.php.
  • Rewrites the "Other bundled-extension hooks" note in docs/hooks-reference.md so it no longer names the removed hooks.
  • Removes the wpdm prefix from the phpcs.xml.dist allowlist. phpcs already excluded /extensions/ and no core PHP uses the prefix.

Left as is: openStationPortal.php uses desktop-mode-cron-manager only as a sample plugin URL, and the comments in src/native-windows.ts and extensions/base/ describe what externally installed copies still do.

Testing instructions

npm run build && npm run lint && npm run typecheck && npm run test:js
npm run lint:php

The first four scripts passed locally (test:js: 543 files, 6,899 tests). lint:php needs its wp-env container, so phpcs -n ran directly over the same tree instead and was clean.

Open WordPress Playground Preview

@epeicher
epeicher added this pull request to stack #1050 October 23, 2026 16:18
Base automatically changed from remove/code-editor-extension to trunk October 23, 2026 16:22
The Cron Manager lives in AllTerrainDeveloper/desktop-mode-official-extensions,
so drop the bundled copy, its capability-gate test, and the docs and lint
references that pointed at it.
@epeicher
epeicher force-pushed the remove/cron-manager-extension branch from 38a85c0 to 652accf Compare October 23, 2026 16:22
@epeicher
epeicher merged commit 680f4f6 into trunk Oct 23, 2026
7 checks passed
@epeicher
epeicher deleted the remove/cron-manager-extension branch October 23, 2026 16:37
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.

1 participant