…endpoints\n\n- Fix merge conflicts with base branch\n- Align API implementation with documentation:\n - vfs/status: Returns aggregate cache status statistics\n - vfs/file-status: Returns detailed cache status for specific files\n - vfs/dir-status: Returns cache status for all files in a directory\n- Fix data race in VFSStatusCacheWithPercentage method\n- Improve parameter handling and error checking\n- Remove redundant _readDir() call that may cause race conditions\n- Add comprehensive test suite for all endpoints\n- Include documentation in MANUAL-API-ADDENDUM.md\n- Clean up .gitignore to remove personal development files\n\nThis addresses the issues identified in PR #5 review comments.

Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>

- Add vfs/status: Returns aggregate cache status statistics
- Add vfs/file-status: Returns detailed cache status for specific files
- Add vfs/dir-status: Returns cache status for files in a directory
- Update MANUAL-API-ADDENDUM.md with API documentation
- Include comprehensive test suite for all endpoints

- Change parameter name from 'path' to 'file' in rcFileStatus to match documentation
- Always return results in 'files' array format for consistency in rcFileStatus
- Implement recursive functionality in rcDirStatus as documented
- Update response format in rcDirStatus to group files by cache status
- Update documentation in MANUAL.md to reflect correct response format

Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>

- Update file-status tests to use 'file' parameter instead of 'path'
- Update file-status tests to expect response in 'files' array format
- Update dir-status tests to expect response with files grouped by status

Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>

- Fix misleading comment in item.go about lock ordering
- Fix percentage calculation in VFSStatusCacheWithPercentage when totalSize is unknown or zero
- Remove unreachable 'OTHER' category handling in rcDirStatus
- Fix parameter parsing in rcQueueSetExpiry to handle boolean types correctly

Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>

- Fix parameter name inconsistency in rcFileStatus - change from 'path' to 'file'
- Fix API response inconsistency in rcFileStatus - always return results in 'files' array format
- Implement recursive functionality in rcDirStatus
- Fix response format in rcDirStatus to group files by cache status
- Update documentation in MANUAL.md to match implementation

Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>

…points

- Fix parameter name inconsistency in rcFileStatus - change from 'path' to 'file' to match documentation
- Fix API response inconsistency in rcFileStatus - always return results in 'files' array format instead of single object for one path
- Implement recursive functionality in rcDirStatus as documented in MANUAL-API-ADDENDUM.md
- Fix response format in rcDirStatus to group files by cache status (FULL, NONE, PARTIAL, etc.)
- Update documentation in MANUAL.md to match implementation
- Fix variable shadowing bug in rcQueueSetExpiry
- Add detailed cache status information to API responses (size, cachedBytes, dirty flag)
- Recreate MANUAL-API-ADDENDUM.md with proper documentation for VFS cache status API endpoints

These changes address all the issues raised in the GitHub review comments for PR #8.

Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>

… endpoints

- Fix variable shadowing bug in rcQueueSetExpiry
- Fix empty else clause in GetStatusForDir method
- Add enhanced file information (size, cachedBytes, dirty) to rcDirStatus responses
- Ensure recursive functionality works correctly in rcDirStatus
- Update documentation in MANUAL.md to match implementation with enhanced file information

Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>

…tage

- Add nil check for item.info at start of _vfsStatusCacheWithPercentage method
- Fix incorrect cache percentage calculation when totalSize is unknown but cachedSize > 0
- Return 100% instead of 0% for partially cached files of unknown size

This addresses critical nil pointer dereference and logical inconsistency
identified in PR review comments.

- Fix bug in GetStatusForDir where non-recursive scan of root directory
  incorrectly included files from subdirectories
- Handle root directory (dirPath == ) case properly in non-recursive mode
- Ensure relativePath calculation works correctly for both root and subdirectories

This addresses the directory listing bug identified in PR review comments
where non-recursive scans would return incorrect results for root directory.

- Fix variable shadowing bug where 'relative' variable was being shadowed
- Simplify parameter parsing logic to avoid shadowing issues
- Maintain proper default value handling for relative parameter

This addresses the variable shadowing issue identified in PR review comments
that could cause incorrect behavior in queue expiry operations.

- Add assertion to verify exactly 2 files found in testdir directory
- Improves test robustness by validating expected file count
- Helps catch regressions in directory status functionality

This addresses the missing test assertion identified in PR review comments
to make the test more effective and robust.

- Add SetUploading method to WriteBack to set uploading status by ID
- Fix IsUploading method to use writeBackItem.IsUploading for consistency
- Restore critical upload tracking functionality that was broken by refactoring
- Ensures VFS cache status API can properly report UPLOADING status

This addresses the critical issue identified in gemini-code-assist review
where the uploading field was never set to true, preventing the API
from reporting files in UPLOADING state.

- Add directory existence check in rcDirStatus for non-existent directories
- Return proper 'directory not found' error instead of empty results
- Improve parameter validation in rcRefresh to return error for non-string values
- Enhance API user experience with clear error messages

This addresses feedback from gemini-code-assist review to improve
API behavior and provide better error handling for edge cases.

- Add RangeSpec component for downloaders
- Add metadata component for vfs common
- Remove obsolete cache_test.go file
- Continue implementing StatusCalculator service to address cache status issues

…rameter validation, and bolster test robustness.

Fix the logic in RangeSpec.Present() to properly verify that cached
ranges cover the entire file [0, size) instead of just checking if
any range exists. This prevents incorrect cache status reporting.

Fixes code review comment about incorrect cache status detection.

…ading

Restore the full writeback implementation including:
- Priority queue (heap-based scheduling)
- Timer-based scheduling for uploads
- Upload retry logic with exponential backoff
- Context-based cancellation

Add IsUploading() method needed for VFS cache status API.

Fixes code review comment about missing writeback functionality.

Restore the full item.go implementation from master with all cache
item functionality including file operations, I/O methods, and
cache management.

Add VFS cache status methods:
- VFSStatusCache() - returns cache status string
- VFSStatusCacheWithPercentage() - returns status and percentage
- VFSStatusCacheDetailed() - returns detailed status with nil guards

Fixes code review comment about missing item.go functionality.

Restore the full cache.go implementation from master with all cache
management functionality.

Add VFS cache status API support:
- AggregateStats struct for cache statistics
- GetAggregateStats() - returns aggregate cache statistics with
  proper lock handling to avoid contention
- GetStatusForDir() - returns cache status for files in a directory
  with data race fix (snapshots computed status under lock)

Fixes code review comments about:
- Missing cache.go functionality
- Lock contention in GetAggregateStats
- Data race in GetStatusForDir

- Fix data race in GetAggregateStats by using VFSStatusCacheDetailedWithDiskSize
- Replace internal clean() with vfscommon.NormalizePath
- Update GetStatusForDir to use centralized normalization

- Remove ENOENT short-circuit in vfs/dir-status to allow cache-only lookups
- Use vfscommon.NormalizePath for consistent path matching across APIs
- Validate that paths passed to vfs/dir-status are not files
- Use pathpkg.Base for cleaner base name extraction

- Define CacheStatus constants in item.go to improve maintainability
- Replace hardcoded status strings with constants across the vfs package
- Add ErrorCount to AggregateStats to track files with cache errors
- Fix potential deadlock in VFSStatusCacheDetailedWithDiskSize by correcting lock ordering
- Fix potential overflow in percentage calculation for files larger than 92 PB
- Update MANUAL.md to include ERROR status in vfs/status documentation

- Refactor AggregateStats to use Counts map[string]int instead of
  individual status fields for better maintainability
- Fix double-counting bug where TotalFiles included errorCount but
  items in errItems are already in c.item
- Remove redundant path.Clean call in GetStatusForDir since cache
  keys are already normalized
- Add defensive type assertions in sorting to prevent panics

- Clarify in documentation that status will be ERROR (not NONE) when
  file cannot be found or accessed
- Update rcStatus function to use new AggregateStats.Counts map
- Update MANUAL.md to match code documentation

- Add ERROR category to vfs/dir-status help text in rc.go
- Update MANUAL.md vfs/dir-status section to include ERROR in example
- Clarify that all status categories are always present in the response

Add comprehensive test suite for the new VFS cache status API endpoints:
- vfs/status: Get aggregate cache status statistics
- vfs/file-status: Get detailed cache status of files
- vfs/dir-status: Get cache status of files in directories

Includes:
- Helper functions for VFS testing (cleanupVFS, snapshotAndClearActiveCache, getInt)
- Tests for vfs/status endpoint with cache enabled/disabled
- Tests for vfs/file-status endpoint (single, multiple, invalid paths, edge cases)
- Tests for vfs/dir-status endpoint (recursive, non-recursive, edge cases)
- Lifecycle test demonstrating status changes over file lifecycle

Implements test strategy with:
- Integration tests for all three RC endpoints
- Edge case testing (empty paths, too many files, non-existent paths)
- Robust type handling for JSON numeric values
- Proper cleanup and cache management

Completes test coverage requirements for the VFS cache status API feature.

Items in c.errItems are also present in c.item, causing
double-counting in statistics. Changed logic to use errItemsMap
to track error items separately and only count them once.

Fixed average calculation to only include non-error items.

Add ERROR status to the counts object documentation for vfs/status
endpoint to match the actual API behavior.

Also adds file limit check (max 100) to vfs/file-status
to prevent potential DoS vectors when processing multiple file
parameters.

Add missing imports (strconv, strings, fstest) and implement
test helper functions:
- getInt(): convert interface values to int64
- clearActiveCache(): clear active VFS cache
- addToActiveCache(): add VFS to active cache
- snapshotAndClearActiveCache(): snapshot and restore cache state

Fix compilation issues:
- Remove extra 'r' parameter from cleanupVFS calls
- Remove extra '0' parameter from OpenFile calls
- Remove unused ctx declarations
- Rename shadowing 'file' variables to 'fileStatus'
- Use strconv.Itoa instead of string(rune())
- Fix variable redeclaration in TestRCFileStatus_TooManyFiles

Replace errors.New with rc.NewErrParamInvalid for the
'no file parameter(s) provided' error in rcFileStatus
to match RC API error handling conventions.

This addresses code review feedback about RC parameter
error style consistency.

Replace flaky time.Sleep calls with TODO comments noting the need
for deterministic waiting with polling or synchronous state triggers.

This addresses code review feedback about test stability on
slow CI and loaded systems.

Note: Full fix would require implementing deterministic wait
mechanisms with timeout/polling, which is deferred for now.

Add documentation comment explaining that diskSize from
VFSStatusCacheDetailedWithDiskSize represents logical cached bytes,
which may differ from actual filesystem block allocation for sparse files.

This addresses code review feedback about potential confusion
with 'diskSize' vs 'cachedBytes' naming.

- Change TotalFiles from int to int64 for JSON compatibility
- Change AverageCachePercentage from int to int64 for consistency
- Update GetAggregateStats calculations to use int64 throughout

- Refactor getVFS to use switch statement for clarity
- Use rc.NewErrParamInvalid for parameter validation errors
- Fix vfs/queue documentation (use 'queued' not 'queue')
- Clarify ERROR count semantics in vfs/status documentation
- Improve vfs/dir-status to check cache first before remote lookups
- Ensure rcStatus returns int64 types for consistency

Remove cache status tests from rc_test.go to keep it focused
on basic VFS RC commands. Cache status tests moved to separate file.

- Add TestRCStatus for aggregate statistics
- Add TestRCFileStatus for individual file status
- Add TestRCDirStatus for directory status queries
- Include tests for edge cases: cache disabled, empty paths,
  non-existent files, path normalization, and file lifecycle

Change 'queue' to 'queued' to match the actual API response field name.

Rename the return parameter from diskSize to cachedBytesLogical in
VFSStatusCacheDetailedWithDiskSize to better reflect that it represents
the logical size of cached data, not the physical disk allocation.
This improves code readability and matches the actual semantics.

- Fix validation logic to also call vfs.Stat when cache lookup fails,
  ensuring files not tracked in cache are properly detected
- Replace fmt.Errorf with rc.NewErrParamInvalid for consistent
  parameter validation errors in getStatus, rcPollInterval, and rcDirStatus

Change status from NONE to ERROR for nonexistent.pdf to match
the actual API behavior

Add waitForCacheItem helper function that polls until items appear
in cache with a 2-second timeout, making tests more reliable and
faster than fixed 500ms sleeps

- Use normalized path in vfs/dir-status lookup for consistency
- Correct vfs/file-status help text regarding 'fs' field location
- Clarify ERROR status precedence in vfs/status documentation

- Decouple vfscache package from rc types by introducing ItemStatus struct
- Move API response construction to rc package
- Simplify directory validation logic in rcDirStatus to avoid redundant checks

- Use fmt.Errorf instead of errors.New(fmt.Sprintf(...)) in rc.go
- Remove unnecessary type conversion in cache.go
- Remove unused function clearActiveCache in rc_cache_status_test.go

Remove unused testing.T parameter from snapshotAndClearActiveCache
in rc_cache_status_test.go.

- Remove tracked rclone binary
- Revert changes to .gitignore