feat: revert index.ts changes and implement queryPermission method

Co-authored-by: streamich <[email protected]>

Author: Copilot

src/fsa/CoreFileSystemDirectoryHandle.ts

@@ -15,6 +15,7 @@ import type {
   GetFileHandleOptions,
   IFileSystemDirectoryHandle,
   IFileSystemFileHandle,
+  IFileSystemHandle,
   RemoveEntryOptions,
 } from './types';
 import type { Superblock } from '../core/Superblock';
@@ -27,7 +28,7 @@ import { MODE, FLAGS } from '../node/constants';
  * @see https://developer.mozilla.org/en-US/docs/Web/API/FileSystemDirectoryHandle
  */
 export class CoreFileSystemDirectoryHandle extends CoreFileSystemHandle implements IFileSystemDirectoryHandle {
-  protected readonly ctx: Partial<CoreFsaContext>;
+  protected readonly ctx: CoreFsaContext;
   /** Directory path with trailing slash. */
   public readonly __path: string;
 
@@ -36,8 +37,9 @@ export class CoreFileSystemDirectoryHandle extends CoreFileSystemHandle implemen
     path: string,
     ctx: Partial<CoreFsaContext> = {},
   ) {
-    super('directory', basename(path, ctx.separator || '/'));
-    this.ctx = createCtx(ctx);
+    const fullCtx = createCtx(ctx);
+    super('directory', basename(path, fullCtx.separator), fullCtx);
+    this.ctx = fullCtx;
     this.__path = path[path.length - 1] === this.ctx.separator ? path : path + this.ctx.separator;
   }
 
@@ -242,7 +244,7 @@ export class CoreFileSystemDirectoryHandle extends CoreFileSystemHandle implemen
    * @param possibleDescendant The {@link CoreFileSystemHandle} from which
    *        to return the relative path.
    */
-  public async resolve(possibleDescendant: CoreFileSystemHandle): Promise<string[] | null> {
+  public async resolve(possibleDescendant: IFileSystemHandle): Promise<string[] | null> {
     if (
       possibleDescendant instanceof CoreFileSystemDirectoryHandle ||
       possibleDescendant instanceof CoreFileSystemFileHandle

src/fsa/CoreFileSystemFileHandle.ts

@@ -20,9 +20,9 @@ export class CoreFileSystemFileHandle extends CoreFileSystemHandle implements IF
     public readonly __path: string,
     ctx: Partial<CoreFsaContext> = {},
   ) {
-    ctx = createCtx(ctx);
-    super('file', basename(__path, ctx.separator!));
-    this.ctx = ctx as CoreFsaContext;
+    const fullCtx = createCtx(ctx);
+    super('file', basename(__path, fullCtx.separator), fullCtx);
+    this.ctx = fullCtx;
   }
 
   /**

src/fsa/CoreFileSystemHandle.ts

@@ -1,5 +1,5 @@
 import { CorePermissionStatus } from './CorePermissionStatus';
-import type { IFileSystemHandle, FileSystemHandlePermissionDescriptor } from './types';
+import type { IFileSystemHandle, FileSystemHandlePermissionDescriptor, CoreFsaContext } from './types';
 
 /**
  * Represents a File System Access API file handle `FileSystemHandle` object,
@@ -8,10 +8,15 @@ import type { IFileSystemHandle, FileSystemHandlePermissionDescriptor } from './
  * @see [MDN Documentation](https://developer.mozilla.org/en-US/docs/Web/API/FileSystemHandle)
  */
 export abstract class CoreFileSystemHandle implements IFileSystemHandle {
+  protected readonly ctx: CoreFsaContext;
+
   constructor(
     public readonly kind: 'file' | 'directory',
     public readonly name: string,
-  ) {}
+    ctx: CoreFsaContext,
+  ) {
+    this.ctx = ctx;
+  }
 
   /**
    * Compares two handles to see if the associated entries (either a file or directory) match.
@@ -30,7 +35,17 @@ export abstract class CoreFileSystemHandle implements IFileSystemHandle {
   public queryPermission(
     fileSystemHandlePermissionDescriptor: FileSystemHandlePermissionDescriptor,
   ): CorePermissionStatus {
-    throw new Error('Not implemented');
+    // Check if the requested mode is compatible with the context mode
+    const requestedMode = fileSystemHandlePermissionDescriptor.mode;
+    const contextMode = this.ctx.mode;
+
+    // If requesting readwrite but context only allows read, deny
+    if (requestedMode === 'readwrite' && contextMode === 'read') {
+      return new CorePermissionStatus('denied', requestedMode);
+    }
+
+    // Otherwise grant the permission
+    return new CorePermissionStatus('granted', requestedMode);
   }
 
   /**

src/fsa/__tests__/CoreFileSystemHandle.test.ts

@@ -52,10 +52,35 @@ onlyOnNode20('CoreFileSystemHandle', () => {
     expect(file.isSameEntry(folder)).toBe(false);
   });
 
-  test('queryPermission throws not implemented error', async () => {
+  test('queryPermission returns permission status based on context mode', async () => {
     const { dir } = setup({ 'test.txt': 'content' });
     const file = await dir.getFileHandle('test.txt');
-    expect(() => file.queryPermission({ mode: 'read' })).toThrow('Not implemented');
+
+    // Test read permission request (should be granted since context allows readwrite)
+    const readPermission = file.queryPermission({ mode: 'read' });
+    expect(readPermission.state).toBe('granted');
+    expect(readPermission.name).toBe('read');
+
+    // Test readwrite permission request (should be granted since context allows readwrite)
+    const readwritePermission = file.queryPermission({ mode: 'readwrite' });
+    expect(readwritePermission.state).toBe('granted');
+    expect(readwritePermission.name).toBe('readwrite');
+  });
+
+  test('queryPermission denies readwrite when context only allows read', async () => {
+    const core = Superblock.fromJSON({ 'test.txt': 'content' }, '/');
+    const dir = new CoreFileSystemDirectoryHandle(core, '/', { mode: 'read' });
+    const file = await dir.getFileHandle('test.txt');
+
+    // Test read permission request (should be granted)
+    const readPermission = file.queryPermission({ mode: 'read' });
+    expect(readPermission.state).toBe('granted');
+    expect(readPermission.name).toBe('read');
+
+    // Test readwrite permission request (should be denied since context only allows read)
+    const readwritePermission = file.queryPermission({ mode: 'readwrite' });
+    expect(readwritePermission.state).toBe('denied');
+    expect(readwritePermission.name).toBe('readwrite');
   });
 
   test('requestPermission throws not implemented error', async () => {

src/index.ts

@@ -7,29 +7,6 @@ import type { FsPromisesApi } from './node/types';
 import type * as misc from './node/types/misc';
 import { fsSynchronousApiList } from './node/lists/fsSynchronousApiList';
 import { fsCallbackApiList } from './node/lists/fsCallbackApiList';
-
-// Re-export core FSA implementation
-export {
-  CoreFileSystemHandle,
-  CoreFileSystemDirectoryHandle,
-  CoreFileSystemFileHandle,
-  CoreFileSystemSyncAccessHandle,
-  CoreFileSystemWritableFileStream,
-  CorePermissionStatus,
-  coreToFsa,
-  type CoreFsaContext,
-} from './fsa';
-
-// Re-export node-to-fsa implementation
-export {
-  NodeFileSystemHandle,
-  NodeFileSystemDirectoryHandle,
-  NodeFileSystemFileHandle,
-  nodeToFsa,
-  type NodeFsaContext,
-  type NodeFsaFs,
-} from './node-to-fsa';
-
 const { F_OK, R_OK, W_OK, X_OK } = constants;
 
 export { DirectoryJSON, NestedDirectoryJSON, Volume };