@nocobase/app-template-default 1.0.0-beta.81 → 1.0.0-beta.82

package/tests/client/role-return.test.tsx

@@ -0,0 +1,104 @@
+import { answerApi, renderWithApp } from '@nocobase/app-testing/client';
+import { apiClientToken, type ClientApplication } from '@nocobase/app-client';
+import {
+  AuthorizationClient,
+  authorizationClientToken,
+} from '@nocobase/app-plugin-authorization/client';
+import { screen, waitFor } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
+import { Outlet, Routes, Route, useLocation } from 'react-router';
+import { expect, it, vi } from 'vitest';
+import RoleForm from '../../client/pages/settings/role-form.js';
+import enUS from '../../client/locales/en-US.js';
+import zhCN from '../../client/locales/zh-CN.js';
+
+const namespace = '@nocobase/role-return-test';
+function Parent() {
+  return <Outlet context={{ reload: () => undefined }} />;
+}
+function Location() {
+  const { pathname, search } = useLocation();
+  return (
+    <output data-testid='location'>
+      {pathname}
+      {search}
+    </output>
+  );
+}
+it.each(
+  ['/roles', '/administration/roles'].flatMap((parent) =>
+    ['back', 'cancel', 'save'].map((action) => ({ parent, action })),
+  ),
+)('returns to $parent on $action', async ({ parent, action }) => {
+  const calls = vi.fn(({ path }: { path: string }) => {
+    if (path === 'authorization/permissions')
+      return { data: { unrestricted: true, permissions: [] } };
+    if (path.endsWith('/options'))
+      return {
+        data: {
+          sections: [],
+          recordAccess: [],
+          subjectTypes: [],
+          collections: [],
+        },
+      };
+    return {
+      data: { key: 'sales', title: 'Sales', grants: [], revision: 'one' },
+    };
+  });
+  const result = await renderWithApp(
+    <>
+      <Routes>
+        <Route path={parent} element={<Parent />}>
+          <Route index element={<h1>Role list</h1>} />
+          <Route path=':key/edit' element={<RoleForm />} />
+        </Route>
+      </Routes>
+      <Location />
+    </>,
+    {
+      route: `${parent}/sales/edit?q=Sales&page=2`,
+      namespace,
+      namespaces: {
+        [namespace]: {
+          'en-US': async () => ({ default: enUS }),
+          'zh-CN': async () => ({ default: zhCN }),
+        },
+      },
+      services: (app: ClientApplication) =>
+        app.container.instance(
+          authorizationClientToken,
+          new AuthorizationClient(app.container.resolve(apiClientToken)),
+        ),
+      fetch: answerApi(calls),
+    },
+  );
+  expect(
+    await screen.findByRole('link', { name: enUS.settings.roles }),
+  ).toHaveAttribute('href', `${parent}?q=Sales&page=2`);
+  if (action === 'back') {
+    await userEvent.click(
+      screen.getByRole('link', { name: enUS.settings.roles }),
+    );
+  } else if (action === 'save') {
+    const name = await screen.findByRole('textbox', { name: /Role name/ });
+    await userEvent.type(name, ' updated');
+    await userEvent.click(
+      screen.getByRole('button', { name: 'Save', exact: true }),
+    );
+    await waitFor(() =>
+      expect(calls).toHaveBeenCalledWith(
+        expect.objectContaining({ method: 'PATCH' }),
+      ),
+    );
+  } else {
+    await userEvent.click(
+      await screen.findByRole('button', { name: 'Cancel' }),
+    );
+  }
+  await screen.findByRole('heading', { name: 'Role list' });
+  expect(screen.getByTestId('location')).toHaveTextContent(
+    `${parent}?q=Sales&page=2`,
+  );
+  result.unmount();
+});

package/tests/project/internal-packages.test.ts

@@ -0,0 +1,98 @@
+import { spawnSync } from 'node:child_process';
+import {
+  cpSync,
+  existsSync,
+  mkdirSync,
+  mkdtempSync,
+  readFileSync,
+  rmSync,
+  writeFileSync,
+} from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+import { expect, it } from 'vitest';
+
+import packageMetadata from '../../package.json' with { type: 'json' };
+
+it('checks every internal package without selecting the root application and propagates a failed check', () => {
+  const root = mkdtempSync(path.join(os.tmpdir(), 'app-internal-checks-'));
+  try {
+    cpSync(
+      path.resolve(import.meta.dirname, '../../scripts'),
+      path.join(root, 'scripts'),
+      { recursive: true },
+    );
+    writeFileSync(
+      path.join(root, 'pnpm-workspace.yaml'),
+      "packages: ['packages/*']\nverifyDepsBeforeRun: false\n",
+    );
+    writeFileSync(
+      path.join(root, 'package.json'),
+      JSON.stringify({
+        name: 'fixture-app',
+        private: true,
+        scripts: {
+          'check:packages': packageMetadata.scripts['check:packages'],
+          check: 'node -e "process.exit(99)"',
+        },
+      }),
+    );
+    for (const name of ['first', 'second']) {
+      const directory = path.join(root, 'packages', name);
+      mkdirSync(directory, { recursive: true });
+      writeFileSync(
+        path.join(directory, 'package.json'),
+        JSON.stringify({
+          name: `fixture-${name}`,
+          private: true,
+          scripts: { check: 'node check.cjs' },
+        }),
+      );
+      writeFileSync(
+        path.join(directory, 'check.cjs'),
+        "require('node:fs').writeFileSync('checked', 'yes');\n",
+      );
+    }
+    const run = () =>
+      spawnSync('pnpm', ['check:packages'], {
+        cwd: root,
+        encoding: 'utf8',
+        timeout: 30_000,
+      });
+    expect(run().status).toBe(0);
+    for (const name of ['first', 'second']) {
+      expect(
+        readFileSync(path.join(root, 'packages', name, 'checked'), 'utf8'),
+      ).toBe('yes');
+    }
+    writeFileSync(
+      path.join(root, 'packages/second/check.cjs'),
+      'process.exit(7);\n',
+    );
+    expect(run().status).not.toBe(0);
+    rmSync(path.join(root, 'packages/first/checked'));
+    writeFileSync(
+      path.join(root, 'packages/second/package.json'),
+      JSON.stringify({ name: 'fixture-second', scripts: { check: '  ' } }),
+    );
+    const missing = run();
+    expect(missing.status).not.toBe(0);
+    expect(missing.stderr).toContain('packages/second');
+    expect(existsSync(path.join(root, 'packages/first/checked'))).toBe(false);
+    writeFileSync(
+      path.join(root, 'packages/second/package.json'),
+      JSON.stringify({ name: 'fixture-second' }),
+    );
+    expect(run().status).not.toBe(0);
+    // pnpm workspace exclusions apply to both the preflight and the script run.
+    writeFileSync(
+      path.join(root, 'pnpm-workspace.yaml'),
+      "packages: ['packages/*', '!packages/second']\nverifyDepsBeforeRun: false\n",
+    );
+    expect(run().status).toBe(0);
+    rmSync(path.join(root, 'packages'), { recursive: true });
+    expect(run().status).toBe(0);
+  } finally {
+    rmSync(root, { recursive: true, force: true });
+  }
+}, 60_000);

package/tests/project/tailwind-sources.test.ts

@@ -101,7 +101,7 @@ describe('tailwind content sources', () => {
     }
   });
 
-  it('scans canonical Registry source exposed by a plugin', () => {
+  it('ignores obsolete plugin-owned Registry recipes', () => {
     const store = mkdtempSync(path.join(tmpdir(), 'nb3-tailwind-registry-'));
     try {
       const registry = path.join(
@@ -124,7 +124,7 @@ describe('tailwind content sources', () => {
 
       const files = contentFilesIn(store);
 
-      expect(files).toContain(path.join(realpathSync(registry), 'demo.tsx'));
+      expect(files).toEqual([]);
     } finally {
       rmSync(store, { recursive: true, force: true });
     }

package/client/extensions/nocobase-file-component-ui/README.md

@@ -1,13 +0,0 @@
-# File Component UI
-
-This Registry item installs editable App source for upload, list, thumbnail, and preview controls. The default destination is client/extensions/nocobase-file-component-ui. Import components and UI types from its index.ts.
-
-Pass the public ClientFileRepository returned by the App's clientFileRepositoryManagerToken to FileUploadField as repository. Read-only components consume FileRecord arrays and their contentUrl directly; they require no repository prop. See the Registry components section in the plugin's SKILL.md for installation, dependencies, and a complete React example.
-
-Upload success is controlled by value/onChange. Temporary progress and error items stay internal. Use onStatusChange to block form submission while uploading or after a failure. Upload cancellation forwards AbortSignal; it does not guarantee rollback after a server commit. removeOnDelete removes metadata only. App-owned business relations and physical cleanup remain the App's responsibility.
-
-Preview handles safe raster images, PDF, audio/video and text/Markdown. The content route serves attachments, so PDF is fetched and embedded as a local blob retyped to `application/pdf`; a response carrying an active markup type is reported as a preview failure rather than embedded, because a blob URL is same-origin. DOCX/XLSX/PPTX render locally with lazily loaded `@silurus/ooxml`; legacy Office and OpenDocument formats retain Office Online, which requires an internet-accessible absolute URL and cannot use the App session. Active HTML/SVG/XML and unsafe URL schemes are rejected. Markdown does not execute raw HTML.
-
-Follow the plugin Skill's Registry installation and preview sections for the exact viewer dependency, Vite dependency-prebundling exclusion, authentication adapter boundaries and acceptance checks. Same-origin content fetches use session cookies; external fetches omit credentials and require CORS. Bearer-only access requires changes to the App-owned source; setting `contentUrl` to an arbitrary blob URL is not supported. OOXML failures show an error and an optional download action; `download={false}` suppresses that action.
-
-There is no public/private record flag or access-token endpoint. The item contains no extension.ts, route, collection, or Server implementation. A private disk does not authorize a content request. Installed copies belong to the App; review upgrades with a three-way merge and preserve customizations. Supply translated labels and adapt additional copy in installed source.