fix(filesystem): treat "." as root equivalent in CompositeFilesystem

[chickenlj]

Summary

• WorkspacePathNormalizer.tryStrip() returns "." when the input path equals the workspace root prefix, but CompositeFilesystem.ls(), grep(), and glob() only merged routed virtual directories (e.g. skills/, memory/, knowledge/) when the path was exactly "/".
• This caused list_files at the workspace root to return "Empty or not a directory", making the agent unable to browse the workspace.
• Fix: treat "." as equivalent to "/" in all three methods.

Fixes #1737

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
.../harness/agent/filesystem/CompositeFilesystem.java	0.00%	3 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

[AgentScopeJavaBot]

🤖 AI Review

This is a correct and well-scoped fix for #1737. WorkspacePathNormalizer.tryStrip() returns "." when the input path equals the workspace root, but CompositeFilesystem.ls(), grep(), and glob() only treated "/" as root, causing workspace-root listing to fail. The fix adds "." as a root-equivalent path in all three methods. One inconsistency is worth addressing: ls() normalizes "." to "/" before delegating to the default backend, but grep() and glob() pass the raw "." through — this works with LocalFilesystem but would break with RemoteFilesystem (whose normalizePath(".") yields "/.") and is inconsistent within the same class. No tests are added for the regression scenario.

(inline comments could not be attached — line numbers fell outside PR hunks. See archived report.)

[AgentScopeJavaBot]

🤖 AI Review

This is a correct and well-scoped fix for #1737. WorkspacePathNormalizer.tryStrip() returns "." when the input path equals the workspace root, but CompositeFilesystem.ls(), grep(), and glob() only treated "/" as root, causing workspace-root listing to fail. The fix adds "." as a root-equivalent path in all three methods. One inconsistency is worth addressing: ls() normalizes "." to "/" before delegating to the default backend, but grep() and glob() pass the raw "." through — this works with LocalFilesystem but would break with RemoteFilesystem (whose normalizePath(".") yields "/.") and is inconsistent within the same class. No tests are added for the regression scenario.

(inline comments could not be attached — line numbers fell outside PR hunks. See archived report.)