Skip to content

fix: use relative path instead absolute path in file change notifications#1892

Merged
tusharmath merged 3 commits intomainfrom
feat/use-absolute-paths-in-file-change-notification
Nov 12, 2025
Merged

fix: use relative path instead absolute path in file change notifications#1892
tusharmath merged 3 commits intomainfrom
feat/use-absolute-paths-in-file-change-notification

Conversation

@laststylebender14
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the type: feature Brand new functionality, features, pages, workflows, endpoints, etc. label Nov 12, 2025
@laststylebender14 laststylebender14 marked this pull request as ready for review November 12, 2025 05:32
@tusharmath tusharmath changed the title feat: use relative path instead absolute path in file change notifications fix: use relative path instead absolute path in file change notifications Nov 12, 2025
@github-actions github-actions Bot added the type: fix Iterations on existing features or infrastructure. label Nov 12, 2025
@tusharmath tusharmath enabled auto-merge (squash) November 12, 2025 09:32
@tusharmath tusharmath removed the type: feature Brand new functionality, features, pages, workflows, endpoints, etc. label Nov 12, 2025
@tusharmath tusharmath merged commit fbd422d into main Nov 12, 2025
9 checks passed
@tusharmath tusharmath deleted the feat/use-absolute-paths-in-file-change-notification branch November 12, 2025 09:33
dariuszkowalski-com added a commit to dariuszkowalski-com/forge that referenced this pull request Nov 26, 2025
- Add _forge_action_editor function to open external editor for command composition
- Use FORGE_EDITOR environment variable with fallback to EDITOR and nano
- Create temporary file in /tmp with automatic cleanup
- Insert editor content into shell buffer with : prefix
- Remove default content for clean prompt experience
- Handle editor errors and empty content gracefully
- Add 'edit' and 'ed' command aliases in forge-accept-line

Resolves tailcallhq#1892

Co-authored-by: ForgeCode <noreply@forgecode.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: fix Iterations on existing features or infrastructure.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants