From 9698beff07d1c83d98a36b46c0cec8bcca5d41d2 Mon Sep 17 00:00:00 2001 From: Andrey Sobolev Date: Wed, 12 Nov 2025 21:55:36 +0700 Subject: [PATCH] Enable formatting check Signed-off-by: Andrey Sobolev --- .github/workflows/ci.yml | 10 ++++++++++ packages/middleware/src/spacePermissions.ts | 9 +++++++-- 2 files changed, 17 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 1474647070..1f8853587e 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -25,6 +25,16 @@ jobs: run: node common/scripts/install-run-rush.js install - name: Rush validate run: node common/scripts/install-run-rush.js validate --verbose + - name: Formatting... + run: node common/scripts/install-run-rush.js format --force + - name: Check files formatting + run: | + echo '================================================================' + echo 'Checking for diff files' + echo '================================================================' + git diff '*.js' '*.ts' '*.svelte' '*.json' '*.yaml' | cat + [ -z "$(git diff --name-only '*.js' '*.ts' '*.svelte' '*.json' '*.yaml' | cat)" ] + echo '================================================================' - name: Prepare server run: | cd ./tests diff --git a/packages/middleware/src/spacePermissions.ts b/packages/middleware/src/spacePermissions.ts index b570c8aca7..1a0b1c56b9 100644 --- a/packages/middleware/src/spacePermissions.ts +++ b/packages/middleware/src/spacePermissions.ts @@ -160,10 +160,15 @@ export class SpacePermissionsMiddleware extends BaseMiddleware implements Middle private checkPermission (ctx: MeasureContext, space: Ref, tx: TxCUD): boolean { const account = ctx.contextData.account const permissions = this.permissionsBySpace[space]?.[account.uuid] ?? [] - let withoutMatch: Permission | undefined = undefined + let withoutMatch: Permission | undefined for (const permission of permissions) { if (permission.txClass === undefined || permission.txClass !== tx._class) continue - if (permission.objectClass !== undefined && !this.context.hierarchy.isDerived(tx.objectClass, permission.objectClass)) continue + if ( + permission.objectClass !== undefined && + !this.context.hierarchy.isDerived(tx.objectClass, permission.objectClass) + ) { + continue + } if (permission.txMatch === undefined) { withoutMatch = permission continue