Files
jackwener__opencli/clis/twitter/block.ts
jakevin 4fe9a73ebc refactor: migrate adapter imports to package exports (#795)
* refactor: migrate adapter imports to package exports

Replace all relative imports (../../src/registry.js, ../../browser/cdp.js, etc.)
with package exports (@jackwener/opencli/registry, @jackwener/opencli/errors, etc.)
across all 484 adapter files.

This decouples adapter import resolution from directory structure:
- User CLIs in ~/.opencli/clis/ resolve via node_modules symlink
- Internal adapters resolve via Node.js self-referencing
- No more shim files needed for import resolution

Changes:
- package.json: add sub-path exports for all public modules
- clis/**: replace relative imports with @jackwener/opencli/...
- discovery.ts: simplify ensureUserCliCompatShims to symlink-only
- registry-api.ts: export CommandArgs type
- Remove root-level shim directories (browser/, download/, pipeline/)
- Remove shim entries from tsconfig.json include and package.json files

* test: add regression tests for package exports

Prevents regressions like #788/#791 by:
1. Scanning all adapter files for forbidden relative imports
   (../../src/, ../../browser/, etc.) — fails if any remain
2. Verifying every package.json export maps to an existing source file

18 new test cases.

* fix: use junction on Windows + broaden test patterns

- discovery.ts: use 'junction' symlink type on Windows (no admin required)
- package-exports.test.ts: generalize forbidden patterns to catch any
  depth of ../ traversal (not just ../../ and ../../../)

* fix: update stale vi.mock/importActual paths in adapter tests

Test files still used old relative paths for vi.mock() and
vi.importActual() calls. Updated 5 test files to use package exports.
Also broadened regression test patterns to catch mock/importActual paths.

* fix: use rm instead of unlink for symlink cleanup, add warn on failure

Addresses review feedback from Astro-Han:
- rm() handles both symlinks and stale directories (unlink fails on dirs)
- Log a warning when symlink creation fails instead of silent catch

* docs: update import examples to use package exports

Update all documentation, contributing guides, and skills to use
@jackwener/opencli/registry instead of ../../src/registry.js.

Without this, users following the docs would write adapters with
broken imports since the old shim files are no longer created.
2026-04-05 16:02:17 +08:00

94 lines
3.3 KiB
TypeScript

import { CommandExecutionError } from '@jackwener/opencli/errors';
import { cli, Strategy } from '@jackwener/opencli/registry';
import type { IPage } from '@jackwener/opencli/types';
cli({
site: 'twitter',
name: 'block',
description: 'Block a Twitter user',
domain: 'x.com',
strategy: Strategy.UI,
browser: true,
args: [
{ name: 'username', type: 'string', positional: true, required: true, help: 'Twitter screen name (without @)' },
],
columns: ['status', 'message'],
func: async (page: IPage | null, kwargs: any) => {
if (!page) throw new CommandExecutionError('Browser session required for twitter block');
const username = kwargs.username.replace(/^@/, '');
await page.goto(`https://x.com/${username}`);
await page.wait({ selector: '[data-testid="primaryColumn"]' });
const result = await page.evaluate(`(async () => {
try {
let attempts = 0;
// Check if already blocked (profile shows "Blocked" / unblock button)
while (attempts < 20) {
const blockedIndicator = document.querySelector('[data-testid$="-unblock"]');
if (blockedIndicator) {
return { ok: true, message: 'Already blocking @${username}.' };
}
const moreBtn = document.querySelector('[data-testid="userActions"]');
if (moreBtn) break;
await new Promise(r => setTimeout(r, 500));
attempts++;
}
const moreBtn = document.querySelector('[data-testid="userActions"]');
if (!moreBtn) {
return { ok: false, message: 'Could not find user actions menu. Are you logged in?' };
}
// Open the more actions menu
moreBtn.click();
await new Promise(r => setTimeout(r, 1000));
// Find the Block menu item
const menuItems = document.querySelectorAll('[role="menuitem"]');
let blockItem = null;
for (const item of menuItems) {
if (item.textContent && item.textContent.includes('Block')) {
blockItem = item;
break;
}
}
if (!blockItem) {
return { ok: false, message: 'Could not find Block option in menu.' };
}
blockItem.click();
await new Promise(r => setTimeout(r, 1000));
// Confirm the block in the dialog
const confirmBtn = document.querySelector('[data-testid="confirmationSheetConfirm"]');
if (confirmBtn) {
confirmBtn.click();
await new Promise(r => setTimeout(r, 1500));
}
// Verify
const verify = document.querySelector('[data-testid$="-unblock"]');
if (verify) {
return { ok: true, message: 'Successfully blocked @${username}.' };
} else {
return { ok: false, message: 'Block action initiated but UI did not update.' };
}
} catch (e) {
return { ok: false, message: e.toString() };
}
})()`);
if (result.ok) await page.wait(2);
return [{
status: result.ok ? 'success' : 'failed',
message: result.message
}];
}
});