Compare commits
6 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 5a6d9a2575 | |||
| 7a7ee041a3 | |||
| 0dc4eaba52 | |||
| 31ab561859 | |||
| a5b4fedf9f | |||
| b84a50f87d |
26
changelog.md
26
changelog.md
@@ -1,5 +1,31 @@
|
|||||||
# Changelog
|
# Changelog
|
||||||
|
|
||||||
|
## 2025-12-08 - 5.1.0 - feat(watchers)
|
||||||
|
Improve write stabilization and ignore temporary editor files
|
||||||
|
|
||||||
|
- Add maxWaitTime option (ms) to IWatcherOptions and WriteStabilizer to cap how long stabilization will wait (default: 1000ms).
|
||||||
|
- WriteStabilizer: reduce default stabilityThreshold from 300ms to 100ms and track write start time to enforce maxWaitTime and avoid indefinite polling.
|
||||||
|
- Node and Deno watchers: detect and ignore common temporary/editor files (e.g. *.tmp.*, *.swp, *.swx, trailing ~, .#*) to prevent spurious events from atomic saves.
|
||||||
|
- Node watcher: treat rename (atomic-save) events as already-complete files and emit add/change immediately without stabilization.
|
||||||
|
- Deno watcher: use the configured maxWaitTime and polling-based stabilization for modify events to ensure consistent behavior across runtimes.
|
||||||
|
|
||||||
|
## 2025-11-30 - 5.0.0 - BREAKING CHANGE(@push.rocks/smartwatch)
|
||||||
|
Rename package and update branding/docs: switch from @push.rocks/smartchok to @push.rocks/smartwatch, update repository/homepage/bugs URLs and author, and refresh README examples and install instructions.
|
||||||
|
|
||||||
|
- Package name changed from @push.rocks/smartchok to @push.rocks/smartwatch in package.json
|
||||||
|
- Repository, homepage and issue URLs updated to point to the new smartwatch repository
|
||||||
|
- Author changed to Task Venture Capital GmbH in package metadata
|
||||||
|
- README updated: install commands, import examples and references now use @push.rocks/smartwatch
|
||||||
|
- Documentation text and branding updated throughout README (project name, internal references)
|
||||||
|
|
||||||
|
## 2025-11-30 - 4.0.1 - fix(readme)
|
||||||
|
Update README: refine description and clarify trademark/legal information
|
||||||
|
|
||||||
|
- Refined project description and tagline for clarity and brevity (adds lightweight wording and an emoji).
|
||||||
|
- Updated Trademark section to explicitly mention third-party trademarks, add guidance about usage and approval, and clarify that trademarks are not covered by the MIT license.
|
||||||
|
- Minor legal/company wording and formatting fixes (e.g. 'District Court' capitalization and contact sentence tweaks).
|
||||||
|
- General README wording and formatting improvements for clearer instructions and feature descriptions.
|
||||||
|
|
||||||
## 2025-11-30 - 4.0.0 - BREAKING CHANGE(watchers)
|
## 2025-11-30 - 4.0.0 - BREAKING CHANGE(watchers)
|
||||||
Replace chokidar with native platform watchers and add cross-runtime support (Node.js, Deno, Bun); introduce write stabilization and internal glob matching
|
Replace chokidar with native platform watchers and add cross-runtime support (Node.js, Deno, Bun); introduce write stabilization and internal glob matching
|
||||||
|
|
||||||
|
|||||||
12
package.json
12
package.json
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "@push.rocks/smartchok",
|
"name": "@push.rocks/smartwatch",
|
||||||
"version": "4.0.0",
|
"version": "5.1.0",
|
||||||
"private": false,
|
"private": false,
|
||||||
"description": "A cross-runtime file watcher with glob pattern support for Node.js, Deno, and Bun.",
|
"description": "A cross-runtime file watcher with glob pattern support for Node.js, Deno, and Bun.",
|
||||||
"main": "dist_ts/index.js",
|
"main": "dist_ts/index.js",
|
||||||
@@ -13,14 +13,14 @@
|
|||||||
},
|
},
|
||||||
"repository": {
|
"repository": {
|
||||||
"type": "git",
|
"type": "git",
|
||||||
"url": "https://code.foss.global/push.rocks/smartchok.git"
|
"url": "https://code.foss.global/push.rocks/smartwatch.git"
|
||||||
},
|
},
|
||||||
"author": "Lossless GmbH",
|
"author": "Task Venture Capital GmbH",
|
||||||
"license": "MIT",
|
"license": "MIT",
|
||||||
"bugs": {
|
"bugs": {
|
||||||
"url": "https://gitlab.com/push.rocks/smartchok/issues"
|
"url": "https://gitlab.com/push.rocks/smartwatch/issues"
|
||||||
},
|
},
|
||||||
"homepage": "https://code.foss.global/push.rocks/smartchok",
|
"homepage": "https://code.foss.global/push.rocks/smartwatch",
|
||||||
"engines": {
|
"engines": {
|
||||||
"node": ">=20.0.0"
|
"node": ">=20.0.0"
|
||||||
},
|
},
|
||||||
|
|||||||
104
readme.md
104
readme.md
@@ -1,6 +1,6 @@
|
|||||||
# @push.rocks/smartchok
|
# @push.rocks/smartwatch
|
||||||
|
|
||||||
A cross-runtime file watcher with glob pattern support for **Node.js**, **Deno**, and **Bun**. Built with native file watching APIs for maximum performance and zero heavyweight dependencies.
|
A lightweight, cross-runtime file watcher with glob pattern support for **Node.js**, **Deno**, and **Bun**. Zero heavyweight dependencies — just native file watching APIs for maximum performance. 🚀
|
||||||
|
|
||||||
## Issue Reporting and Security
|
## Issue Reporting and Security
|
||||||
|
|
||||||
@@ -9,27 +9,28 @@ For reporting bugs, issues, or security vulnerabilities, please visit [community
|
|||||||
## Install
|
## Install
|
||||||
|
|
||||||
```sh
|
```sh
|
||||||
npm install @push.rocks/smartchok
|
npm install @push.rocks/smartwatch
|
||||||
# or
|
# or
|
||||||
pnpm add @push.rocks/smartchok
|
pnpm add @push.rocks/smartwatch
|
||||||
```
|
```
|
||||||
|
|
||||||
## Features
|
## Features
|
||||||
|
|
||||||
🌐 **Cross-Runtime** - Works on Node.js 20+, Deno, and Bun
|
🌐 **Cross-Runtime** — Works seamlessly on Node.js 20+, Deno, and Bun
|
||||||
🔍 **Glob Pattern Support** - Watch files using familiar patterns like `**/*.ts`
|
🔍 **Glob Pattern Support** — Watch files using familiar patterns like `**/*.ts` and `src/**/*.{js,jsx}`
|
||||||
📡 **RxJS Observables** - Subscribe to file system events using reactive streams
|
📡 **RxJS Observables** — Subscribe to file system events using reactive streams
|
||||||
🔄 **Dynamic Watching** - Add or remove watch patterns at runtime
|
🔄 **Dynamic Watching** — Add or remove watch patterns at runtime
|
||||||
⚡ **Native Performance** - Uses `fs.watch()` and `Deno.watchFs()` directly
|
⚡ **Native Performance** — Uses `fs.watch()` on Node.js/Bun and `Deno.watchFs()` on Deno
|
||||||
🎯 **TypeScript First** - Full TypeScript support with comprehensive type definitions
|
✨ **Write Stabilization** — Built-in debouncing prevents duplicate events during file writes
|
||||||
📦 **Minimal Dependencies** - No heavyweight watcher libraries required
|
🎯 **TypeScript First** — Full TypeScript support with comprehensive type definitions
|
||||||
|
📦 **Minimal Footprint** — No chokidar, no FSEvents bindings — just ~500 lines of focused code
|
||||||
|
|
||||||
## Usage
|
## Usage
|
||||||
|
|
||||||
### Basic Setup
|
### Basic Setup
|
||||||
|
|
||||||
```typescript
|
```typescript
|
||||||
import { Smartwatch } from '@push.rocks/smartchok';
|
import { Smartwatch } from '@push.rocks/smartwatch';
|
||||||
|
|
||||||
// Create a watcher with glob patterns
|
// Create a watcher with glob patterns
|
||||||
const watcher = new Smartwatch([
|
const watcher = new Smartwatch([
|
||||||
@@ -52,6 +53,7 @@ const changeObservable = await watcher.getObservableFor('change');
|
|||||||
changeObservable.subscribe({
|
changeObservable.subscribe({
|
||||||
next: ([path, stats]) => {
|
next: ([path, stats]) => {
|
||||||
console.log(`File changed: ${path}`);
|
console.log(`File changed: ${path}`);
|
||||||
|
console.log(`New size: ${stats?.size} bytes`);
|
||||||
},
|
},
|
||||||
error: (err) => {
|
error: (err) => {
|
||||||
console.error(`Error: ${err}`);
|
console.error(`Error: ${err}`);
|
||||||
@@ -74,10 +76,10 @@ unlinkObservable.subscribe(([path]) => {
|
|||||||
### Supported Events
|
### Supported Events
|
||||||
|
|
||||||
| Event | Description |
|
| Event | Description |
|
||||||
|-------|-------------|
|
|-------------|--------------------------------|
|
||||||
| `add` | File has been added |
|
| `add` | File has been added |
|
||||||
| `addDir` | Directory has been added |
|
| `addDir` | Directory has been added |
|
||||||
| `change` | File has been changed |
|
| `change` | File has been modified |
|
||||||
| `unlink` | File has been removed |
|
| `unlink` | File has been removed |
|
||||||
| `unlinkDir` | Directory has been removed |
|
| `unlinkDir` | Directory has been removed |
|
||||||
| `error` | Error occurred |
|
| `error` | Error occurred |
|
||||||
@@ -108,7 +110,7 @@ await watcher.stop();
|
|||||||
### Complete Example
|
### Complete Example
|
||||||
|
|
||||||
```typescript
|
```typescript
|
||||||
import { Smartwatch } from '@push.rocks/smartchok';
|
import { Smartwatch } from '@push.rocks/smartwatch';
|
||||||
|
|
||||||
async function watchProject() {
|
async function watchProject() {
|
||||||
// Initialize with patterns
|
// Initialize with patterns
|
||||||
@@ -153,29 +155,38 @@ watchProject();
|
|||||||
|
|
||||||
## How It Works
|
## How It Works
|
||||||
|
|
||||||
smartchok uses native file watching APIs for each runtime:
|
smartwatch uses native file watching APIs for each runtime:
|
||||||
|
|
||||||
| Runtime | API Used |
|
| Runtime | API Used |
|
||||||
|---------|----------|
|
|-----------------|----------------------------------|
|
||||||
| **Node.js 20+** | `fs.watch({ recursive: true })` |
|
| **Node.js 20+** | `fs.watch({ recursive: true })` |
|
||||||
| **Deno** | `Deno.watchFs()` |
|
| **Deno** | `Deno.watchFs()` |
|
||||||
| **Bun** | Node.js compatibility layer |
|
| **Bun** | Node.js compatibility layer |
|
||||||
|
|
||||||
Since native APIs don't support glob patterns, smartchok handles pattern matching internally using [picomatch](https://github.com/micromatch/picomatch):
|
### Under the Hood
|
||||||
|
|
||||||
1. **Base path extraction** - Extracts the static base path from each glob pattern
|
Native file watching APIs don't support glob patterns directly, so smartwatch handles pattern matching internally:
|
||||||
2. **Efficient watching** - Native watchers monitor the base directories
|
|
||||||
3. **Pattern filtering** - Events are filtered through picomatch matchers before being emitted
|
1. **Base path extraction** — Extracts the static portion from each glob pattern (e.g., `./src/` from `./src/**/*.ts`)
|
||||||
|
2. **Efficient watching** — Native watchers monitor only the base directories
|
||||||
|
3. **Pattern filtering** — Events are filtered through [picomatch](https://github.com/micromatch/picomatch) matchers before emission
|
||||||
|
4. **Event deduplication** — Built-in throttling prevents duplicate events from rapid file operations
|
||||||
|
|
||||||
### Write Stabilization
|
### Write Stabilization
|
||||||
|
|
||||||
smartchok includes built-in write stabilization (similar to chokidar's `awaitWriteFinish`). When a file is being written, events are held until the file size stabilizes, preventing multiple events for a single write operation.
|
smartwatch includes built-in write stabilization (similar to chokidar's `awaitWriteFinish`). When a file is being written, events are held until the file size stabilizes, preventing multiple events for a single write operation.
|
||||||
|
|
||||||
|
Default settings:
|
||||||
|
- **Stability threshold**: 300ms
|
||||||
|
- **Poll interval**: 100ms
|
||||||
|
|
||||||
## Requirements
|
## Requirements
|
||||||
|
|
||||||
- **Node.js 20+** - Required for native recursive watching
|
| Runtime | Version |
|
||||||
- **Deno** - Any version with `Deno.watchFs()` support
|
|-----------------|----------------------------------------|
|
||||||
- **Bun** - Uses Node.js compatibility
|
| **Node.js** | 20+ (required for native recursive watching) |
|
||||||
|
| **Deno** | Any version with `Deno.watchFs()` support |
|
||||||
|
| **Bun** | Uses Node.js compatibility |
|
||||||
|
|
||||||
## API Reference
|
## API Reference
|
||||||
|
|
||||||
@@ -189,37 +200,62 @@ new Smartwatch(patterns: string[])
|
|||||||
|
|
||||||
Creates a new Smartwatch instance with the given glob patterns.
|
Creates a new Smartwatch instance with the given glob patterns.
|
||||||
|
|
||||||
|
**Parameters:**
|
||||||
|
- `patterns` — Array of glob patterns to watch (e.g., `['./src/**/*.ts', './config/*.json']`)
|
||||||
|
|
||||||
#### Methods
|
#### Methods
|
||||||
|
|
||||||
| Method | Returns | Description |
|
| Method | Returns | Description |
|
||||||
|--------|---------|-------------|
|
|----------------------------------------|------------------------------------------------|------------------------------------------------|
|
||||||
| `start()` | `Promise<void>` | Starts watching for file changes |
|
| `start()` | `Promise<void>` | Starts watching for file changes |
|
||||||
| `stop()` | `Promise<void>` | Stops the file watcher |
|
| `stop()` | `Promise<void>` | Stops the file watcher and cleans up resources |
|
||||||
| `add(patterns: string[])` | `void` | Adds patterns to watch |
|
| `add(patterns: string[])` | `void` | Adds additional patterns to watch |
|
||||||
| `remove(pattern: string)` | `void` | Removes a pattern from watching |
|
| `remove(pattern: string)` | `void` | Removes a pattern from the watch list |
|
||||||
| `getObservableFor(event: TFsEvent)` | `Promise<Observable<[string, Stats]>>` | Returns an RxJS observable for the specified event |
|
| `getObservableFor(event: TFsEvent)` | `Promise<Observable<[string, Stats]>>` | Returns an RxJS observable for the specified event |
|
||||||
|
|
||||||
#### Properties
|
#### Properties
|
||||||
|
|
||||||
| Property | Type | Description |
|
| Property | Type | Description |
|
||||||
|----------|------|-------------|
|
|----------|-------------------------------------------|--------------------------|
|
||||||
| `status` | `'idle' \| 'starting' \| 'watching'` | Current watcher status |
|
| `status` | `'idle' \| 'starting' \| 'watching'` | Current watcher status |
|
||||||
|
|
||||||
|
### Types
|
||||||
|
|
||||||
|
```typescript
|
||||||
|
type TFsEvent = 'add' | 'addDir' | 'change' | 'error' | 'unlink' | 'unlinkDir' | 'ready' | 'raw';
|
||||||
|
type TSmartwatchStatus = 'idle' | 'starting' | 'watching';
|
||||||
|
```
|
||||||
|
|
||||||
|
## Why smartwatch?
|
||||||
|
|
||||||
|
| Feature | smartwatch | chokidar |
|
||||||
|
|-------------------------|----------------------|--------------------|
|
||||||
|
| Native API | ✅ Direct `fs.watch` | ❌ FSEvents bindings |
|
||||||
|
| Cross-runtime | ✅ Node, Deno, Bun | ❌ Node only |
|
||||||
|
| Dependencies | 4 small packages | ~20 packages |
|
||||||
|
| Write stabilization | ✅ Built-in | ✅ Built-in |
|
||||||
|
| Glob support | ✅ picomatch | ✅ anymatch |
|
||||||
|
| Bundle size | ~15KB | ~200KB+ |
|
||||||
|
|
||||||
|
If you need a lightweight file watcher without native compilation headaches, smartwatch has you covered.
|
||||||
|
|
||||||
## License and Legal Information
|
## License and Legal Information
|
||||||
|
|
||||||
This repository contains open-source code that is licensed under the MIT License. A copy of the MIT License can be found in the [license](license) file within this repository.
|
This repository contains open-source code licensed under the MIT License. A copy of the license can be found in the [LICENSE](./LICENSE) file.
|
||||||
|
|
||||||
**Please note:** The MIT License does not grant permission to use the trade names, trademarks, service marks, or product names of the project, except as required for reasonable and customary use in describing the origin of the work and reproducing the content of the NOTICE file.
|
**Please note:** The MIT License does not grant permission to use the trade names, trademarks, service marks, or product names of the project, except as required for reasonable and customary use in describing the origin of the work and reproducing the content of the NOTICE file.
|
||||||
|
|
||||||
### Trademarks
|
### Trademarks
|
||||||
|
|
||||||
This project is owned and maintained by Task Venture Capital GmbH. The names and logos associated with Task Venture Capital GmbH and any related products or services are trademarks of Task Venture Capital GmbH and are not included within the scope of the MIT license granted herein. Use of these trademarks must comply with Task Venture Capital GmbH's Trademark Guidelines, and any usage must be approved in writing by Task Venture Capital GmbH.
|
This project is owned and maintained by Task Venture Capital GmbH. The names and logos associated with Task Venture Capital GmbH and any related products or services are trademarks of Task Venture Capital GmbH or third parties, and are not included within the scope of the MIT license granted herein.
|
||||||
|
|
||||||
|
Use of these trademarks must comply with Task Venture Capital GmbH's Trademark Guidelines or the guidelines of the respective third-party owners, and any usage must be approved in writing. Third-party trademarks used herein are the property of their respective owners and used only in a descriptive manner, e.g. for an implementation of an API or similar.
|
||||||
|
|
||||||
### Company Information
|
### Company Information
|
||||||
|
|
||||||
Task Venture Capital GmbH
|
Task Venture Capital GmbH
|
||||||
Registered at District court Bremen HRB 35230 HB, Germany
|
Registered at District Court Bremen HRB 35230 HB, Germany
|
||||||
|
|
||||||
For any legal inquiries or if you require further information, please contact us via email at hello@task.vc.
|
For any legal inquiries or further information, please contact us via email at hello@task.vc.
|
||||||
|
|
||||||
By using this repository, you acknowledge that you have read this section, agree to comply with its terms, and understand that the licensing of the code does not imply endorsement by Task Venture Capital GmbH of any derivative works.
|
By using this repository, you acknowledge that you have read this section, agree to comply with its terms, and understand that the licensing of the code does not imply endorsement by Task Venture Capital GmbH of any derivative works.
|
||||||
|
|||||||
@@ -2,7 +2,7 @@
|
|||||||
* autocreated commitinfo by @push.rocks/commitinfo
|
* autocreated commitinfo by @push.rocks/commitinfo
|
||||||
*/
|
*/
|
||||||
export const commitinfo = {
|
export const commitinfo = {
|
||||||
name: '@push.rocks/smartchok',
|
name: '@push.rocks/smartwatch',
|
||||||
version: '4.0.0',
|
version: '5.1.0',
|
||||||
description: 'A cross-runtime file watcher with glob pattern support for Node.js, Deno, and Bun.'
|
description: 'A cross-runtime file watcher with glob pattern support for Node.js, Deno, and Bun.'
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -132,8 +132,9 @@ export class Smartwatch {
|
|||||||
basePaths: watchPaths,
|
basePaths: watchPaths,
|
||||||
depth: 4,
|
depth: 4,
|
||||||
followSymlinks: false,
|
followSymlinks: false,
|
||||||
stabilityThreshold: 300,
|
stabilityThreshold: 100,
|
||||||
pollInterval: 100
|
pollInterval: 100,
|
||||||
|
maxWaitTime: 1000
|
||||||
});
|
});
|
||||||
|
|
||||||
// Subscribe to watcher events and dispatch to appropriate subjects
|
// Subscribe to watcher events and dispatch to appropriate subjects
|
||||||
|
|||||||
@@ -3,6 +3,7 @@ import * as fs from 'fs';
|
|||||||
interface IPendingWrite {
|
interface IPendingWrite {
|
||||||
lastSize: number;
|
lastSize: number;
|
||||||
lastChange: number;
|
lastChange: number;
|
||||||
|
startTime: number;
|
||||||
timeoutId: ReturnType<typeof setTimeout>;
|
timeoutId: ReturnType<typeof setTimeout>;
|
||||||
resolve: (stats: fs.Stats) => void;
|
resolve: (stats: fs.Stats) => void;
|
||||||
reject: (error: Error) => void;
|
reject: (error: Error) => void;
|
||||||
@@ -16,8 +17,9 @@ export class WriteStabilizer {
|
|||||||
private pendingWrites = new Map<string, IPendingWrite>();
|
private pendingWrites = new Map<string, IPendingWrite>();
|
||||||
|
|
||||||
constructor(
|
constructor(
|
||||||
private stabilityThreshold: number = 300,
|
private stabilityThreshold: number = 100,
|
||||||
private pollInterval: number = 100
|
private pollInterval: number = 100,
|
||||||
|
private maxWaitTime: number = 1000
|
||||||
) {}
|
) {}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -28,6 +30,8 @@ export class WriteStabilizer {
|
|||||||
this.cancel(filePath);
|
this.cancel(filePath);
|
||||||
|
|
||||||
return new Promise((resolve, reject) => {
|
return new Promise((resolve, reject) => {
|
||||||
|
const startTime = Date.now();
|
||||||
|
|
||||||
const poll = async () => {
|
const poll = async () => {
|
||||||
try {
|
try {
|
||||||
const stats = await fs.promises.stat(filePath);
|
const stats = await fs.promises.stat(filePath);
|
||||||
@@ -40,6 +44,13 @@ export class WriteStabilizer {
|
|||||||
|
|
||||||
const now = Date.now();
|
const now = Date.now();
|
||||||
|
|
||||||
|
// Check if we've exceeded max wait time - emit with current stats
|
||||||
|
if (now - pending.startTime >= this.maxWaitTime) {
|
||||||
|
this.pendingWrites.delete(filePath);
|
||||||
|
resolve(stats);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
if (stats.size !== pending.lastSize) {
|
if (stats.size !== pending.lastSize) {
|
||||||
// Size changed - file is still being written, reset timer
|
// Size changed - file is still being written, reset timer
|
||||||
pending.lastSize = stats.size;
|
pending.lastSize = stats.size;
|
||||||
@@ -66,7 +77,8 @@ export class WriteStabilizer {
|
|||||||
|
|
||||||
this.pendingWrites.set(filePath, {
|
this.pendingWrites.set(filePath, {
|
||||||
lastSize: -1,
|
lastSize: -1,
|
||||||
lastChange: Date.now(),
|
lastChange: startTime,
|
||||||
|
startTime: startTime,
|
||||||
timeoutId: setTimeout(poll, this.pollInterval),
|
timeoutId: setTimeout(poll, this.pollInterval),
|
||||||
resolve,
|
resolve,
|
||||||
reject
|
reject
|
||||||
|
|||||||
@@ -28,6 +28,7 @@ export const defaultWatcherOptions: IWatcherOptions = {
|
|||||||
basePaths: [],
|
basePaths: [],
|
||||||
depth: 4,
|
depth: 4,
|
||||||
followSymlinks: false,
|
followSymlinks: false,
|
||||||
stabilityThreshold: 300,
|
stabilityThreshold: 100,
|
||||||
pollInterval: 100
|
pollInterval: 100,
|
||||||
|
maxWaitTime: 1000
|
||||||
};
|
};
|
||||||
|
|||||||
@@ -30,6 +30,8 @@ export interface IWatcherOptions {
|
|||||||
stabilityThreshold: number;
|
stabilityThreshold: number;
|
||||||
/** Poll interval for write detection (ms) */
|
/** Poll interval for write detection (ms) */
|
||||||
pollInterval: number;
|
pollInterval: number;
|
||||||
|
/** Maximum time to wait for write stabilization (ms) */
|
||||||
|
maxWaitTime: number;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|||||||
@@ -74,6 +74,19 @@ export class DenoWatcher implements IWatcher {
|
|||||||
|
|
||||||
constructor(private options: IWatcherOptions) {}
|
constructor(private options: IWatcherOptions) {}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Check if a file is a temporary file created by editors
|
||||||
|
*/
|
||||||
|
private isTemporaryFile(filePath: string): boolean {
|
||||||
|
const basename = filePath.split('/').pop() || '';
|
||||||
|
// Editor temp files: *.tmp.*, *.swp, *.swx, *~, .#*
|
||||||
|
if (basename.includes('.tmp.')) return true;
|
||||||
|
if (basename.endsWith('.swp') || basename.endsWith('.swx')) return true;
|
||||||
|
if (basename.endsWith('~')) return true;
|
||||||
|
if (basename.startsWith('.#')) return true;
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
get isWatching(): boolean {
|
get isWatching(): boolean {
|
||||||
return this._isWatching;
|
return this._isWatching;
|
||||||
}
|
}
|
||||||
@@ -162,6 +175,11 @@ export class DenoWatcher implements IWatcher {
|
|||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Skip temporary files created by editors (atomic saves)
|
||||||
|
if (this.isTemporaryFile(filePath)) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
// Throttle duplicate events
|
// Throttle duplicate events
|
||||||
if (!this.shouldEmit(filePath, kind)) {
|
if (!this.shouldEmit(filePath, kind)) {
|
||||||
return;
|
return;
|
||||||
@@ -169,19 +187,15 @@ export class DenoWatcher implements IWatcher {
|
|||||||
|
|
||||||
try {
|
try {
|
||||||
if (kind === 'create') {
|
if (kind === 'create') {
|
||||||
|
// Create events (atomic saves) don't need stabilization - file is already complete
|
||||||
const stats = await this.statSafe(filePath);
|
const stats = await this.statSafe(filePath);
|
||||||
if (stats) {
|
if (stats) {
|
||||||
// Wait for write to stabilize
|
|
||||||
await this.waitForWriteFinish(filePath);
|
|
||||||
const finalStats = await this.statSafe(filePath);
|
|
||||||
|
|
||||||
if (finalStats) {
|
|
||||||
this.watchedFiles.add(filePath);
|
this.watchedFiles.add(filePath);
|
||||||
const eventType: TWatchEventType = finalStats.isDirectory() ? 'addDir' : 'add';
|
const eventType: TWatchEventType = stats.isDirectory() ? 'addDir' : 'add';
|
||||||
this.events$.next({ type: eventType, path: filePath, stats: finalStats });
|
this.events$.next({ type: eventType, path: filePath, stats });
|
||||||
}
|
|
||||||
}
|
}
|
||||||
} else if (kind === 'modify') {
|
} else if (kind === 'modify') {
|
||||||
|
// Modify events are in-place writes - use stabilization
|
||||||
const stats = await this.statSafe(filePath);
|
const stats = await this.statSafe(filePath);
|
||||||
if (stats && !stats.isDirectory()) {
|
if (stats && !stats.isDirectory()) {
|
||||||
// Wait for write to stabilize
|
// Wait for write to stabilize
|
||||||
@@ -212,6 +226,7 @@ export class DenoWatcher implements IWatcher {
|
|||||||
return new Promise((resolve) => {
|
return new Promise((resolve) => {
|
||||||
let lastSize = -1;
|
let lastSize = -1;
|
||||||
let lastChange = Date.now();
|
let lastChange = Date.now();
|
||||||
|
const startTime = Date.now();
|
||||||
|
|
||||||
const poll = async () => {
|
const poll = async () => {
|
||||||
try {
|
try {
|
||||||
@@ -222,6 +237,14 @@ export class DenoWatcher implements IWatcher {
|
|||||||
}
|
}
|
||||||
|
|
||||||
const now = Date.now();
|
const now = Date.now();
|
||||||
|
|
||||||
|
// Check if we've exceeded max wait time - resolve immediately
|
||||||
|
if (now - startTime >= this.options.maxWaitTime) {
|
||||||
|
this.pendingWrites.delete(filePath);
|
||||||
|
resolve();
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
if (stats.size !== lastSize) {
|
if (stats.size !== lastSize) {
|
||||||
lastSize = stats.size;
|
lastSize = stats.size;
|
||||||
lastChange = now;
|
lastChange = now;
|
||||||
|
|||||||
@@ -20,10 +20,24 @@ export class NodeWatcher implements IWatcher {
|
|||||||
constructor(private options: IWatcherOptions) {
|
constructor(private options: IWatcherOptions) {
|
||||||
this.writeStabilizer = new WriteStabilizer(
|
this.writeStabilizer = new WriteStabilizer(
|
||||||
options.stabilityThreshold,
|
options.stabilityThreshold,
|
||||||
options.pollInterval
|
options.pollInterval,
|
||||||
|
options.maxWaitTime
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Check if a file is a temporary file created by editors
|
||||||
|
*/
|
||||||
|
private isTemporaryFile(filePath: string): boolean {
|
||||||
|
const basename = path.basename(filePath);
|
||||||
|
// Editor temp files: *.tmp.*, *.swp, *.swx, *~, .#*
|
||||||
|
if (basename.includes('.tmp.')) return true;
|
||||||
|
if (basename.endsWith('.swp') || basename.endsWith('.swx')) return true;
|
||||||
|
if (basename.endsWith('~')) return true;
|
||||||
|
if (basename.startsWith('.#')) return true;
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
get isWatching(): boolean {
|
get isWatching(): boolean {
|
||||||
return this._isWatching;
|
return this._isWatching;
|
||||||
}
|
}
|
||||||
@@ -114,6 +128,11 @@ export class NodeWatcher implements IWatcher {
|
|||||||
): Promise<void> {
|
): Promise<void> {
|
||||||
const fullPath = path.join(basePath, filename);
|
const fullPath = path.join(basePath, filename);
|
||||||
|
|
||||||
|
// Skip temporary files created by editors (atomic saves)
|
||||||
|
if (this.isTemporaryFile(fullPath)) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
// Throttle duplicate events
|
// Throttle duplicate events
|
||||||
if (!this.shouldEmit(fullPath, eventType)) {
|
if (!this.shouldEmit(fullPath, eventType)) {
|
||||||
return;
|
return;
|
||||||
@@ -132,23 +151,14 @@ export class NodeWatcher implements IWatcher {
|
|||||||
this.events$.next({ type: 'addDir', path: fullPath, stats });
|
this.events$.next({ type: 'addDir', path: fullPath, stats });
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
// Wait for write to stabilize before emitting
|
// Rename events (atomic saves) don't need stabilization - file is already complete
|
||||||
try {
|
|
||||||
const stableStats = await this.writeStabilizer.waitForWriteFinish(fullPath);
|
|
||||||
const wasWatched = this.watchedFiles.has(fullPath);
|
const wasWatched = this.watchedFiles.has(fullPath);
|
||||||
this.watchedFiles.add(fullPath);
|
this.watchedFiles.add(fullPath);
|
||||||
this.events$.next({
|
this.events$.next({
|
||||||
type: wasWatched ? 'change' : 'add',
|
type: wasWatched ? 'change' : 'add',
|
||||||
path: fullPath,
|
path: fullPath,
|
||||||
stats: stableStats
|
stats
|
||||||
});
|
});
|
||||||
} catch {
|
|
||||||
// File was deleted during stabilization
|
|
||||||
if (this.watchedFiles.has(fullPath)) {
|
|
||||||
this.watchedFiles.delete(fullPath);
|
|
||||||
this.events$.next({ type: 'unlink', path: fullPath });
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
// File doesn't exist - it was deleted
|
// File doesn't exist - it was deleted
|
||||||
@@ -162,7 +172,7 @@ export class NodeWatcher implements IWatcher {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
} else if (eventType === 'change') {
|
} else if (eventType === 'change') {
|
||||||
// File was modified
|
// File was modified in-place - use stabilization for streaming writes
|
||||||
if (stats && !stats.isDirectory()) {
|
if (stats && !stats.isDirectory()) {
|
||||||
try {
|
try {
|
||||||
const stableStats = await this.writeStabilizer.waitForWriteFinish(fullPath);
|
const stableStats = await this.writeStabilizer.waitForWriteFinish(fullPath);
|
||||||
|
|||||||
Reference in New Issue
Block a user