@zenfs/core 1.3.2 → 1.3.4

package/dist/backends/overlay.d.ts

@@ -1,7 +1,7 @@
 import type { File } from '../file.js';
 import type { FileSystemMetadata } from '../filesystem.js';
 import { FileSystem } from '../filesystem.js';
-import { Stats } from '../stats.js';
+import type { Stats } from '../stats.js';
 /**
  * Configuration options for OverlayFS instances.
  */

package/dist/backends/port/fs.d.ts

@@ -4,7 +4,7 @@ import { type MountConfiguration } from '../../config.js';
 import { File } from '../../file.js';
 import { FileSystem, type FileSystemMetadata } from '../../filesystem.js';
 import { Async } from '../../mixins/async.js';
-import { Stats, type FileType } from '../../stats.js';
+import { Stats } from '../../stats.js';
 import type { Backend, FilesystemOf } from '../backend.js';
 import * as RPC from './rpc.js';
 type FileMethods = Omit<ExtractProperties<File, (...args: any[]) => Promise<any>>, typeof Symbol.asyncDispose>;
@@ -37,7 +37,6 @@ export declare class PortFile extends File {
     chmodSync(): void;
     utimes(atime: Date, mtime: Date): Promise<void>;
     utimesSync(): void;
-    _setType(type: FileType): Promise<void>;
     _setTypeSync(): void;
     close(): Promise<void>;
     closeSync(): void;

package/dist/backends/port/fs.js

@@ -67,9 +67,6 @@ export class PortFile extends File {
     utimesSync() {
         this._throwNoSync('utimes');
     }
-    _setType(type) {
-        return this.rpc('_setType', type);
-    }
     _setTypeSync() {
         this._throwNoSync('_setType');
     }

package/dist/config.js

@@ -1,5 +1,5 @@
 import { checkOptions, isBackend, isBackendConfig } from './backends/backend.js';
-import { credentials } from './credentials.js';
+import { useCredentials } from './credentials.js';
 import { DeviceFS } from './devices.js';
 import * as cache from './emulation/cache.js';
 import { config } from './emulation/config.js';
@@ -87,7 +87,7 @@ async function mount(path, mount) {
 export async function configure(configuration) {
     const uid = 'uid' in configuration ? configuration.uid || 0 : 0;
     const gid = 'gid' in configuration ? configuration.gid || 0 : 0;
-    Object.assign(credentials, { uid, gid, suid: uid, sgid: gid, euid: uid, egid: gid });
+    useCredentials({ uid, gid });
     cache.stats.isEnabled = configuration.cacheStats ?? false;
     cache.paths.isEnabled = configuration.cachePaths ?? false;
     config.checkAccess = !configuration.disableAccessChecks;

package/dist/credentials.d.ts

@@ -10,9 +10,21 @@ export interface Credentials {
     sgid: number;
     euid: number;
     egid: number;
+    /**
+     * List of group IDs.
+     */
+    groups: number[];
 }
 export declare const credentials: Credentials;
+export interface CredentialInit {
+    uid: number;
+    gid: number;
+    suid?: number;
+    sgid?: number;
+    euid?: number;
+    egid?: number;
+}
 /**
- * @deprecated
+ * Uses credentials from the provided uid and gid.
  */
-export declare const rootCredentials: Credentials;
+export declare function useCredentials(source: CredentialInit): void;

package/dist/credentials.js

@@ -5,15 +5,17 @@ export const credentials = {
     sgid: 0,
     euid: 0,
     egid: 0,
+    groups: [],
 };
 /**
- * @deprecated
+ * Uses credentials from the provided uid and gid.
  */
-export const rootCredentials = {
-    uid: 0,
-    gid: 0,
-    suid: 0,
-    sgid: 0,
-    euid: 0,
-    egid: 0,
-};
+export function useCredentials(source) {
+    Object.assign(credentials, {
+        suid: source.uid,
+        sgid: source.gid,
+        euid: source.uid,
+        egid: source.gid,
+        ...source,
+    });
+}

package/dist/emulation/promises.js

@@ -710,7 +710,7 @@ export async function link(targetPath, linkPath) {
         if (config.checkAccess && !(await stat(dirname(linkPath))).hasAccess(constants.W_OK)) {
             throw ErrnoError.With('EACCES', dirname(linkPath), 'link');
         }
-        if (config.checkAccess && !(await fs.stat(path)).hasAccess(constants.W_OK | constants.R_OK)) {
+        if (config.checkAccess && !(await fs.stat(path)).hasAccess(constants.R_OK)) {
             throw ErrnoError.With('EACCES', path, 'link');
         }
         return await fs.link(path, link.path);
@@ -737,7 +737,7 @@ export async function symlink(target, path, type = 'file') {
         }
         const handle = __addDisposableResource(env_5, await _open(path, 'w+', 0o644, false), true);
         await handle.writeFile(target.toString());
-        await handle.file._setType(constants.S_IFLNK);
+        await handle.file.chmod(constants.S_IFLNK);
     }
     catch (e_5) {
         env_5.error = e_5;

package/dist/emulation/shared.js

@@ -99,7 +99,9 @@ export function fixError(e, paths) {
     try {
         e.message = fixPaths(e.message, paths);
     }
-    catch { }
+    catch {
+        // `message` is read only
+    }
     return e;
 }
 /**

package/dist/emulation/sync.js

@@ -510,7 +510,7 @@ export function linkSync(targetPath, linkPath) {
         throw ErrnoError.With('EXDEV', linkPath, 'link');
     }
     try {
-        if (config.checkAccess && !fs.statSync(path).hasAccess(constants.W_OK)) {
+        if (config.checkAccess && !fs.statSync(path).hasAccess(constants.R_OK)) {
             throw ErrnoError.With('EACCES', path, 'link');
         }
         return fs.linkSync(path, linkPath);
@@ -535,7 +535,7 @@ export function symlinkSync(target, path, type = 'file') {
     }
     writeFileSync(path, target.toString());
     const file = _openSync(path, 'r+', 0o644, false);
-    file._setTypeSync(constants.S_IFLNK);
+    file.chmodSync(constants.S_IFLNK);
 }
 symlinkSync;
 export function readlinkSync(path, options) {

package/dist/emulation/watchers.js

@@ -2,7 +2,7 @@ import { EventEmitter } from 'eventemitter3';
 import { ErrnoError } from '../error.js';
 import { isStatsEqual } from '../stats.js';
 import { normalizePath } from '../utils.js';
-import { dirname, basename } from './path.js';
+import { basename, dirname } from './path.js';
 import { statSync } from './sync.js';
 /**
  * Base class for file system watchers.
@@ -130,22 +130,22 @@ export function removeWatcher(path, watcher) {
     }
 }
 export function emitChange(eventType, filename) {
-    let normalizedFilename = normalizePath(filename);
+    filename = normalizePath(filename);
     // Notify watchers on the specific file
-    if (watchers.has(normalizedFilename)) {
-        for (const watcher of watchers.get(normalizedFilename)) {
+    if (watchers.has(filename)) {
+        for (const watcher of watchers.get(filename)) {
             watcher.emit('change', eventType, basename(filename));
         }
     }
     // Notify watchers on parent directories if they are watching recursively
-    let parent = dirname(normalizedFilename);
-    while (parent !== normalizedFilename && parent !== '/') {
+    let parent = filename, normalizedFilename;
+    while (parent !== normalizedFilename) {
+        normalizedFilename = parent;
+        parent = dirname(parent);
         if (watchers.has(parent)) {
             for (const watcher of watchers.get(parent)) {
-                watcher.emit('change', eventType, basename(filename));
+                watcher.emit('change', eventType, filename.slice(parent.length + (parent == '/' ? 0 : 1)));
             }
         }
-        normalizedFilename = parent;
-        parent = dirname(parent);
     }
 }

package/dist/file.d.ts

@@ -1,7 +1,7 @@
 import type { FileReadResult } from 'node:fs/promises';
 import type { FileSystem } from './filesystem.js';
 import './polyfills.js';
-import { Stats, type FileType } from './stats.js';
+import { Stats } from './stats.js';
 /**
     Typescript does not include a type declaration for resizable array buffers.
     It has been standardized into ECMAScript though
@@ -123,16 +123,6 @@ export declare abstract class File<FS extends FileSystem = FileSystem> {
      * Change the file timestamps of the file.
      */
     abstract utimesSync(atime: Date, mtime: Date): void;
-    /**
-     * Set the file type
-     * @internal
-     */
-    abstract _setType(type: FileType): Promise<void>;
-    /**
-     * Set the file type
-     * @internal
-     */
-    abstract _setTypeSync(type: FileType): void;
 }
 /**
  * An implementation of `File` that operates completely in-memory.
@@ -243,8 +233,6 @@ export declare class PreloadFile<FS extends FileSystem> extends File<FS> {
     chownSync(uid: number, gid: number): void;
     utimes(atime: Date, mtime: Date): Promise<void>;
     utimesSync(atime: Date, mtime: Date): void;
-    _setType(type: FileType): Promise<void>;
-    _setTypeSync(type: FileType): void;
 }
 /**
  * For the file systems which do not sync to anything.

package/dist/file.js

@@ -2,7 +2,7 @@ import { config } from './emulation/config.js';
 import { O_APPEND, O_CREAT, O_EXCL, O_RDONLY, O_RDWR, O_SYNC, O_TRUNC, O_WRONLY, S_IFMT, size_max } from './emulation/constants.js';
 import { Errno, ErrnoError } from './error.js';
 import './polyfills.js';
-import { Stats } from './stats.js';
+import { _chown, Stats } from './stats.js';
 const validFlags = ['r', 'r+', 'rs', 'rs+', 'w', 'wx', 'w+', 'wx+', 'a', 'ax', 'a+', 'ax+'];
 export function parseFlag(flag) {
     if (typeof flag === 'number') {
@@ -411,8 +411,8 @@ export class PreloadFile extends File {
             throw ErrnoError.With('EBADF', this.path, 'File.chmod');
         }
         this.dirty = true;
-        this.stats.chmod(mode);
-        if (config.syncImmediately)
+        this.stats.mode = (this.stats.mode & (mode > S_IFMT ? ~S_IFMT : S_IFMT)) | mode;
+        if (config.syncImmediately || mode > S_IFMT)
             await this.sync();
     }
     chmodSync(mode) {
@@ -420,8 +420,8 @@ export class PreloadFile extends File {
             throw ErrnoError.With('EBADF', this.path, 'File.chmod');
         }
         this.dirty = true;
-        this.stats.chmod(mode);
-        if (config.syncImmediately)
+        this.stats.mode = (this.stats.mode & (mode > S_IFMT ? ~S_IFMT : S_IFMT)) | mode;
+        if (config.syncImmediately || mode > S_IFMT)
             this.syncSync();
     }
     async chown(uid, gid) {
@@ -429,7 +429,7 @@ export class PreloadFile extends File {
             throw ErrnoError.With('EBADF', this.path, 'File.chown');
         }
         this.dirty = true;
-        this.stats.chown(uid, gid);
+        _chown(this.stats, uid, gid);
         if (config.syncImmediately)
             await this.sync();
     }
@@ -438,7 +438,7 @@ export class PreloadFile extends File {
             throw ErrnoError.With('EBADF', this.path, 'File.chown');
         }
         this.dirty = true;
-        this.stats.chown(uid, gid);
+        _chown(this.stats, uid, gid);
         if (config.syncImmediately)
             this.syncSync();
     }
@@ -462,22 +462,6 @@ export class PreloadFile extends File {
         if (config.syncImmediately)
             this.syncSync();
     }
-    async _setType(type) {
-        if (this.closed) {
-            throw ErrnoError.With('EBADF', this.path, 'File._setType');
-        }
-        this.dirty = true;
-        this.stats.mode = (this.stats.mode & ~S_IFMT) | type;
-        await this.sync();
-    }
-    _setTypeSync(type) {
-        if (this.closed) {
-            throw ErrnoError.With('EBADF', this.path, 'File._setType');
-        }
-        this.dirty = true;
-        this.stats.mode = (this.stats.mode & ~S_IFMT) | type;
-        this.syncSync();
-    }
 }
 /**
  * For the file systems which do not sync to anything.

package/dist/stats.d.ts

@@ -1,5 +1,4 @@
 import type * as Node from 'node:fs';
-import { type Credentials } from './credentials.js';
 import { S_IFDIR, S_IFLNK, S_IFREG } from './emulation/constants.js';
 /**
  * Indicates the type of a file. Applied to 'mode'.
@@ -43,6 +42,10 @@ export interface StatsLike<T extends number | bigint = number | bigint> {
      * Inode number
      */
     ino: T;
+    /**
+     * Number of hard links
+     */
+    nlink: T;
 }
 /**
  * Provides information about a particular entry in the file system.
@@ -136,28 +139,29 @@ export declare abstract class StatsCommon<T extends number | bigint> implements
      * @internal
      */
     hasAccess(mode: number): boolean;
-    /**
-     * Convert the current stats object into a credentials object
-     * @internal
-     */
-    cred(uid?: number, gid?: number): Credentials;
     /**
      * Change the mode of the file.
      * We use this helper function to prevent messing up the type of the file.
      * @internal
+     * @deprecated This will be removed in the next minor release since it is internal
      */
     chmod(mode: number): void;
     /**
      * Change the owner user/group of the file.
      * This function makes sure it is a valid UID/GID (that is, a 32 unsigned int)
      * @internal
+     * @deprecated This will be removed in the next minor release since it is internal
      */
-    chown(uid: number | bigint, gid: number | bigint): void;
+    chown(uid: number, gid: number): void;
     get atimeNs(): bigint;
     get mtimeNs(): bigint;
     get ctimeNs(): bigint;
     get birthtimeNs(): bigint;
 }
+/**
+ * @hidden @internal
+ */
+export declare function _chown(stats: Partial<StatsLike<number>>, uid: number, gid: number): void;
 /**
  * Implementation of Node's `Stats`.
  *

package/dist/stats.js

@@ -124,7 +124,7 @@ export class StatsCommon {
                 perm |= X_OK;
         }
         // Group permissions
-        if (credentials.gid === this.gid) {
+        if (credentials.gid === this.gid || credentials.groups.includes(Number(this.gid))) {
             if (this.mode & S_IRGRP)
                 perm |= R_OK;
             if (this.mode & S_IWGRP)
@@ -142,24 +142,11 @@ export class StatsCommon {
         // Perform the access check
         return (perm & mode) === mode;
     }
-    /**
-     * Convert the current stats object into a credentials object
-     * @internal
-     */
-    cred(uid = Number(this.uid), gid = Number(this.gid)) {
-        return {
-            uid,
-            gid,
-            suid: Number(this.uid),
-            sgid: Number(this.gid),
-            euid: uid,
-            egid: gid,
-        };
-    }
     /**
      * Change the mode of the file.
      * We use this helper function to prevent messing up the type of the file.
      * @internal
+     * @deprecated This will be removed in the next minor release since it is internal
      */
     chmod(mode) {
         this.mode = this._convert((this.mode & S_IFMT) | mode);
@@ -168,6 +155,7 @@ export class StatsCommon {
      * Change the owner user/group of the file.
      * This function makes sure it is a valid UID/GID (that is, a 32 unsigned int)
      * @internal
+     * @deprecated This will be removed in the next minor release since it is internal
      */
     chown(uid, gid) {
         uid = Number(uid);
@@ -192,6 +180,17 @@ export class StatsCommon {
         return BigInt(this.birthtimeMs) * 1000n;
     }
 }
+/**
+ * @hidden @internal
+ */
+export function _chown(stats, uid, gid) {
+    if (!isNaN(uid) && 0 <= uid && uid < 2 ** 32) {
+        stats.uid = uid;
+    }
+    if (!isNaN(gid) && 0 <= gid && gid < 2 ** 32) {
+        stats.gid = gid;
+    }
+}
 /**
  * Implementation of Node's `Stats`.
  *

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@zenfs/core",
-	"version": "1.3.2",
+	"version": "1.3.4",
 	"description": "A filesystem, anywhere",
 	"funding": {
 		"type": "individual",

package/src/backends/overlay.ts

@@ -5,7 +5,7 @@ import { PreloadFile, parseFlag } from '../file.js';
 import type { FileSystemMetadata } from '../filesystem.js';
 import { FileSystem } from '../filesystem.js';
 import { Mutexed } from '../mixins/mutexed.js';
-import { Stats } from '../stats.js';
+import type { Stats } from '../stats.js';
 import { decodeUTF8, encodeUTF8 } from '../utils.js';
 import type { Backend } from './backend.js';
 /** @internal */

package/src/backends/port/fs.ts

@@ -6,7 +6,7 @@ import { Errno, ErrnoError } from '../../error.js';
 import { File } from '../../file.js';
 import { FileSystem, type FileSystemMetadata } from '../../filesystem.js';
 import { Async } from '../../mixins/async.js';
-import { Stats, type FileType } from '../../stats.js';
+import { Stats } from '../../stats.js';
 import type { Backend, FilesystemOf } from '../backend.js';
 import { InMemory } from '../memory.js';
 import * as RPC from './rpc.js';
@@ -104,10 +104,6 @@ export class PortFile extends File {
 		this._throwNoSync('utimes');
 	}
 
-	public _setType(type: FileType): Promise<void> {
-		return this.rpc('_setType', type);
-	}
-
 	public _setTypeSync(): void {
 		this._throwNoSync('_setType');
 	}

package/src/config.ts

@@ -1,6 +1,6 @@
 import type { Backend, BackendConfiguration, FilesystemOf, SharedConfig } from './backends/backend.js';
 import { checkOptions, isBackend, isBackendConfig } from './backends/backend.js';
-import { credentials } from './credentials.js';
+import { useCredentials } from './credentials.js';
 import { DeviceFS } from './devices.js';
 import * as cache from './emulation/cache.js';
 import { config } from './emulation/config.js';
@@ -188,7 +188,7 @@ export async function configure<T extends ConfigMounts>(configuration: Partial<C
 	const uid = 'uid' in configuration ? configuration.uid || 0 : 0;
 	const gid = 'gid' in configuration ? configuration.gid || 0 : 0;
 
-	Object.assign(credentials, { uid, gid, suid: uid, sgid: gid, euid: uid, egid: gid });
+	useCredentials({ uid, gid });
 
 	cache.stats.isEnabled = configuration.cacheStats ?? false;
 	cache.paths.isEnabled = configuration.cachePaths ?? false;

package/src/credentials.ts

@@ -10,6 +10,10 @@ export interface Credentials {
 	sgid: number;
 	euid: number;
 	egid: number;
+	/**
+	 * List of group IDs.
+	 */
+	groups: number[];
 }
 
 export const credentials: Credentials = {
@@ -19,16 +23,27 @@ export const credentials: Credentials = {
 	sgid: 0,
 	euid: 0,
 	egid: 0,
+	groups: [],
 };
 
+export interface CredentialInit {
+	uid: number;
+	gid: number;
+	suid?: number;
+	sgid?: number;
+	euid?: number;
+	egid?: number;
+}
+
 /**
- * @deprecated
+ * Uses credentials from the provided uid and gid.
  */
-export const rootCredentials: Credentials = {
-	uid: 0,
-	gid: 0,
-	suid: 0,
-	sgid: 0,
-	euid: 0,
-	egid: 0,
-};
+export function useCredentials(source: CredentialInit): void {
+	Object.assign(credentials, {
+		suid: source.uid,
+		sgid: source.gid,
+		euid: source.uid,
+		egid: source.gid,
+		...source,
+	});
+}

package/src/emulation/promises.ts

@@ -471,7 +471,7 @@ export async function unlink(path: fs.PathLike): Promise<void> {
 	path = normalizePath(path);
 	const { fs, path: resolved } = resolveMount(path);
 	try {
-		if (config.checkAccess && !(await (cache.stats.get(path) || fs.stat(resolved)))!.hasAccess(constants.W_OK)) {
+		if (config.checkAccess && !(await (cache.stats.get(path) || fs.stat(resolved))).hasAccess(constants.W_OK)) {
 			throw ErrnoError.With('EACCES', resolved, 'unlink');
 		}
 		await fs.unlink(resolved);
@@ -795,7 +795,7 @@ export async function link(targetPath: fs.PathLike, linkPath: fs.PathLike): Prom
 			throw ErrnoError.With('EACCES', dirname(linkPath), 'link');
 		}
 
-		if (config.checkAccess && !(await fs.stat(path)).hasAccess(constants.W_OK | constants.R_OK)) {
+		if (config.checkAccess && !(await fs.stat(path)).hasAccess(constants.R_OK)) {
 			throw ErrnoError.With('EACCES', path, 'link');
 		}
 		return await fs.link(path, link.path);
@@ -822,7 +822,7 @@ export async function symlink(target: fs.PathLike, path: fs.PathLike, type: fs.s
 
 	await using handle = await _open(path, 'w+', 0o644, false);
 	await handle.writeFile(target.toString());
-	await handle.file._setType(constants.S_IFLNK);
+	await handle.file.chmod(constants.S_IFLNK);
 }
 symlink satisfies typeof promises.symlink;
 

package/src/emulation/shared.ts

@@ -114,7 +114,9 @@ export function fixError<E extends ErrnoError>(e: E, paths: Record<string, strin
 	}
 	try {
 		e.message = fixPaths(e.message, paths);
-	} catch {}
+	} catch {
+		// `message` is read only
+	}
 	return e;
 }
 

package/src/emulation/sync.ts

@@ -524,7 +524,7 @@ export function linkSync(targetPath: fs.PathLike, linkPath: fs.PathLike): void {
 		throw ErrnoError.With('EXDEV', linkPath, 'link');
 	}
 	try {
-		if (config.checkAccess && !fs.statSync(path).hasAccess(constants.W_OK)) {
+		if (config.checkAccess && !fs.statSync(path).hasAccess(constants.R_OK)) {
 			throw ErrnoError.With('EACCES', path, 'link');
 		}
 		return fs.linkSync(path, linkPath);
@@ -550,7 +550,7 @@ export function symlinkSync(target: fs.PathLike, path: fs.PathLike, type: fs.sym
 
 	writeFileSync(path, target.toString());
 	const file = _openSync(path, 'r+', 0o644, false);
-	file._setTypeSync(constants.S_IFLNK);
+	file.chmodSync(constants.S_IFLNK);
 }
 symlinkSync satisfies typeof fs.symlinkSync;
 

package/src/emulation/watchers.ts

@@ -4,7 +4,7 @@ import type * as fs from 'node:fs';
 import { ErrnoError } from '../error.js';
 import { isStatsEqual, type Stats } from '../stats.js';
 import { normalizePath } from '../utils.js';
-import { dirname, basename } from './path.js';
+import { basename, dirname } from './path.js';
 import { statSync } from './sync.js';
 
 /**
@@ -171,23 +171,24 @@ export function removeWatcher(path: string, watcher: FSWatcher) {
 }
 
 export function emitChange(eventType: fs.WatchEventType, filename: string) {
-	let normalizedFilename: string = normalizePath(filename);
+	filename = normalizePath(filename);
 	// Notify watchers on the specific file
-	if (watchers.has(normalizedFilename)) {
-		for (const watcher of watchers.get(normalizedFilename)!) {
+	if (watchers.has(filename)) {
+		for (const watcher of watchers.get(filename)!) {
 			watcher.emit('change', eventType, basename(filename));
 		}
 	}
 
 	// Notify watchers on parent directories if they are watching recursively
-	let parent = dirname(normalizedFilename);
-	while (parent !== normalizedFilename && parent !== '/') {
+	let parent = filename,
+		normalizedFilename;
+	while (parent !== normalizedFilename) {
+		normalizedFilename = parent;
+		parent = dirname(parent);
 		if (watchers.has(parent)) {
 			for (const watcher of watchers.get(parent)!) {
-				watcher.emit('change', eventType, basename(filename));
+				watcher.emit('change', eventType, filename.slice(parent.length + (parent == '/' ? 0 : 1)));
 			}
 		}
-		normalizedFilename = parent;
-		parent = dirname(parent);
 	}
 }

package/src/file.ts

@@ -4,7 +4,7 @@ import { O_APPEND, O_CREAT, O_EXCL, O_RDONLY, O_RDWR, O_SYNC, O_TRUNC, O_WRONLY,
 import { Errno, ErrnoError } from './error.js';
 import type { FileSystem } from './filesystem.js';
 import './polyfills.js';
-import { Stats, type FileType } from './stats.js';
+import { _chown, Stats } from './stats.js';
 
 /**
 	Typescript does not include a type declaration for resizable array buffers. 
@@ -251,18 +251,6 @@ export abstract class File<FS extends FileSystem = FileSystem> {
 	 * Change the file timestamps of the file.
 	 */
 	public abstract utimesSync(atime: Date, mtime: Date): void;
-
-	/**
-	 * Set the file type
-	 * @internal
-	 */
-	public abstract _setType(type: FileType): Promise<void>;
-
-	/**
-	 * Set the file type
-	 * @internal
-	 */
-	public abstract _setTypeSync(type: FileType): void;
 }
 
 /**
@@ -573,8 +561,8 @@ export class PreloadFile<FS extends FileSystem> extends File<FS> {
 			throw ErrnoError.With('EBADF', this.path, 'File.chmod');
 		}
 		this.dirty = true;
-		this.stats.chmod(mode);
-		if (config.syncImmediately) await this.sync();
+		this.stats.mode = (this.stats.mode & (mode > S_IFMT ? ~S_IFMT : S_IFMT)) | mode;
+		if (config.syncImmediately || mode > S_IFMT) await this.sync();
 	}
 
 	public chmodSync(mode: number): void {
@@ -582,8 +570,8 @@ export class PreloadFile<FS extends FileSystem> extends File<FS> {
 			throw ErrnoError.With('EBADF', this.path, 'File.chmod');
 		}
 		this.dirty = true;
-		this.stats.chmod(mode);
-		if (config.syncImmediately) this.syncSync();
+		this.stats.mode = (this.stats.mode & (mode > S_IFMT ? ~S_IFMT : S_IFMT)) | mode;
+		if (config.syncImmediately || mode > S_IFMT) this.syncSync();
 	}
 
 	public async chown(uid: number, gid: number): Promise<void> {
@@ -591,7 +579,7 @@ export class PreloadFile<FS extends FileSystem> extends File<FS> {
 			throw ErrnoError.With('EBADF', this.path, 'File.chown');
 		}
 		this.dirty = true;
-		this.stats.chown(uid, gid);
+		_chown(this.stats, uid, gid);
 		if (config.syncImmediately) await this.sync();
 	}
 
@@ -600,7 +588,7 @@ export class PreloadFile<FS extends FileSystem> extends File<FS> {
 			throw ErrnoError.With('EBADF', this.path, 'File.chown');
 		}
 		this.dirty = true;
-		this.stats.chown(uid, gid);
+		_chown(this.stats, uid, gid);
 		if (config.syncImmediately) this.syncSync();
 	}
 
@@ -623,24 +611,6 @@ export class PreloadFile<FS extends FileSystem> extends File<FS> {
 		this.stats.mtime = mtime;
 		if (config.syncImmediately) this.syncSync();
 	}
-
-	public async _setType(type: FileType): Promise<void> {
-		if (this.closed) {
-			throw ErrnoError.With('EBADF', this.path, 'File._setType');
-		}
-		this.dirty = true;
-		this.stats.mode = (this.stats.mode & ~S_IFMT) | type;
-		await this.sync();
-	}
-
-	public _setTypeSync(type: FileType): void {
-		if (this.closed) {
-			throw ErrnoError.With('EBADF', this.path, 'File._setType');
-		}
-		this.dirty = true;
-		this.stats.mode = (this.stats.mode & ~S_IFMT) | type;
-		this.syncSync();
-	}
 }
 
 /**

package/src/stats.ts

@@ -1,5 +1,5 @@
 import type * as Node from 'node:fs';
-import { credentials, type Credentials } from './credentials.js';
+import { credentials } from './credentials.js';
 import {
 	R_OK,
 	S_IFBLK,
@@ -67,6 +67,10 @@ export interface StatsLike<T extends number | bigint = number | bigint> {
 	 * Inode number
 	 */
 	ino: T;
+	/**
+	 * Number of hard links
+	 */
+	nlink: T;
 }
 
 /**
@@ -258,7 +262,7 @@ export abstract class StatsCommon<T extends number | bigint> implements Node.Sta
 		}
 
 		// Group permissions
-		if (credentials.gid === this.gid) {
+		if (credentials.gid === this.gid || credentials.groups.includes(Number(this.gid))) {
 			if (this.mode & S_IRGRP) perm |= R_OK;
 			if (this.mode & S_IWGRP) perm |= W_OK;
 			if (this.mode & S_IXGRP) perm |= X_OK;
@@ -273,25 +277,11 @@ export abstract class StatsCommon<T extends number | bigint> implements Node.Sta
 		return (perm & mode) === mode;
 	}
 
-	/**
-	 * Convert the current stats object into a credentials object
-	 * @internal
-	 */
-	public cred(uid: number = Number(this.uid), gid: number = Number(this.gid)): Credentials {
-		return {
-			uid,
-			gid,
-			suid: Number(this.uid),
-			sgid: Number(this.gid),
-			euid: uid,
-			egid: gid,
-		};
-	}
-
 	/**
 	 * Change the mode of the file.
 	 * We use this helper function to prevent messing up the type of the file.
 	 * @internal
+	 * @deprecated This will be removed in the next minor release since it is internal
 	 */
 	public chmod(mode: number): void {
 		this.mode = this._convert((this.mode & S_IFMT) | mode);
@@ -301,8 +291,9 @@ export abstract class StatsCommon<T extends number | bigint> implements Node.Sta
 	 * Change the owner user/group of the file.
 	 * This function makes sure it is a valid UID/GID (that is, a 32 unsigned int)
 	 * @internal
+	 * @deprecated This will be removed in the next minor release since it is internal
 	 */
-	public chown(uid: number | bigint, gid: number | bigint): void {
+	public chown(uid: number, gid: number): void {
 		uid = Number(uid);
 		gid = Number(gid);
 		if (!isNaN(uid) && 0 <= uid && uid < 2 ** 32) {
@@ -330,6 +321,18 @@ export abstract class StatsCommon<T extends number | bigint> implements Node.Sta
 	}
 }
 
+/**
+ * @hidden @internal
+ */
+export function _chown(stats: Partial<StatsLike<number>>, uid: number, gid: number) {
+	if (!isNaN(uid) && 0 <= uid && uid < 2 ** 32) {
+		stats.uid = uid;
+	}
+	if (!isNaN(gid) && 0 <= gid && gid < 2 ** 32) {
+		stats.gid = gid;
+	}
+}
+
 /**
  * Implementation of Node's `Stats`.
  *

package/tests/fs/watch.test.ts

@@ -115,6 +115,27 @@ suite('Watch Features', () => {
 			resolve();
 		})();
 
+		await fs.promises.unlink(tempFile);
+		await promise;
+	});
+	test('fs.promises.watch should detect file creations recursively', async () => {
+		const rootDir = '/';
+		const subDir = `${testDir}sub-dir`;
+		const tempFile = `${subDir}/tempFile.txt`;
+		await fs.promises.mkdir(subDir);
+		const watcher = fs.promises.watch(rootDir);
+
+		await fs.promises.writeFile(tempFile, 'Temporary content');
+		const { promise, resolve } = Promise.withResolvers<void>();
+		(async () => {
+			for await (const event of watcher) {
+				assert.equal(event.eventType, 'rename');
+				assert.equal(event.filename, tempFile.substring(rootDir.length));
+				break;
+			}
+			resolve();
+		})();
+
 		await fs.promises.unlink(tempFile);
 		await promise;
 	});