Skip to content

fix: Windows clipboard paste and companion path handling - #102

Open
braincoded wants to merge 1 commit into
petrowsky:mainfrom
braincoded:fix/windows-paste-and-companion
Open

braincoded wants to merge 1 commit into
petrowsky:mainfrom
braincoded:fix/windows-paste-and-companion

Conversation

@braincoded

Copy link
Copy Markdown

Summary

  • Make shipped agentic-fm scripts Windows-clipboard-safe by using ASCII <> instead of Unicode ≠ (avoids mojibake that FileMaker wraps in /* ... */ and silently disables).
  • Improve Get agentic-fm path to clear a bad stored path and re-prompt for the folder instead of only showing an error.
  • Make Explode XML fall back to manual /C:/... → /c/... conversion when ConvertFromFileMakerPath(...; 1) returns ? on Windows.
  • Harden the companion for native Windows: Git-Bash-friendly path translation, Desktop auto-detect without requiring WSL, broader repo_path_override handling, and UTF-8 subprocess I/O for fmparse/fmcontext.
  • Update docs / automation.json.example for Windows overrides; add native Windows clipboard support to clipboard.py.

Test plan

  • On Windows: python3 agent/scripts/clipboard.py write filemaker/agentic-fm.xml and paste the agentic-fm folder into a solution; confirm If calculations use <> and are not commented out
  • Run Get agentic-fm path, select the repo root; confirm AGENTIC.FM is valid JSON with a trailing slash path
  • Deliberately set a bad AGENTIC.FM path, re-run Get agentic-fm path, confirm it re-prompts
  • With companion running and automation.json overrides set, run Explode XML on Windows and confirm Finished (no export_file_path is '?')
  • On macOS: smoke-test clipboard write/paste and Explode XML still work

Replace Unicode inequality with ASCII <> in shipped agentic-fm scripts so Windows clipboard pastes do not mojibake calculations into commented-out formulas. Improve Get agentic-fm path to re-prompt on invalid folders, make Explode XML fall back when ConvertFromFileMakerPath returns ?, and teach the companion native Windows Desktop/repo overrides plus UTF-8 subprocess I/O.

Co-authored-by: Cursor <cursoragent@cursor.com>
@braincoded

Copy link
Copy Markdown
Author

@petrowsky Could you take a look when you have a moment? This hardens Windows clipboard paste (Unicode ≠ was getting mojibaked into commented-out calcs) and native Windows companion path handling for Explode XML. Details and test plan are in the PR description / UPDATES.md.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant