Skip to content

Worker commands for files and file-contents#1955

Merged
vigoo merged 1 commit intogolemcloud:mainfrom
Nanashi-lab:workerfiles
Oct 18, 2025
Merged

Worker commands for files and file-contents#1955
vigoo merged 1 commit intogolemcloud:mainfrom
Nanashi-lab:workerfiles

Conversation

@Nanashi-lab
Copy link
Copy Markdown
Contributor

@Nanashi-lab Nanashi-lab commented Aug 18, 2025 •
edited
Loading

/closes #1922
/claim #1922

Unrelated minor - Fixed naming mismatch in Makefile, and add wit/deps to .gitignore in 2 test components

The two changes I made since last time are
files command - File now shows human readable last modified over UNIX time

Listing files for worker test:sc/debug at path /in/
+---------+-----------+-------------+------+---------------------+
| Name    | Kind      | Permissions | Size | Last Modified       |
+---------+-----------+-------------+------+---------------------+
| baz.txt | file      | read-only   |   12 | 2025-08-18 05:45:16 |
+---------+-----------+-------------+------+---------------------+
| inner   | directory | -           |    0 | 2025-08-18 05:45:16 |

file-contents has overwrite confirm with y/N option for the user to interact with

golem-dev worker file-contents test:sc/debug foo.txt
Selected profile: local
Downloading file from worker test:sc/debug at path foo.txt
> File foo.txt already exists. Do you want to overwrite it? Yes
File saved to foo.txt

@Nanashi-lab Nanashi-lab changed the title Worker commands for files and get-contents Worker commands for files and filecontents Aug 18, 2025
@Nanashi-lab Nanashi-lab changed the title Worker commands for files and filecontents Worker commands for files and file-contents Aug 18, 2025
@Nanashi-lab Nanashi-lab marked this pull request as ready for review August 18, 2025 05:47
@Nanashi-lab Nanashi-lab marked this pull request as draft August 18, 2025 07:42
@Nanashi-lab Nanashi-lab marked this pull request as ready for review August 18, 2025 08:38
@Nanashi-lab Nanashi-lab force-pushed the workerfiles branch 2 times, most recently from e9e86d9 to fa6e3df Compare September 7, 2025 08:32
@vigoo
Copy link
Copy Markdown
Contributor

vigoo commented Oct 18, 2025

@Nanashi-lab could you remove the all/main.wits from the PR? Otherwise looks good!

@Nanashi-lab
Copy link
Copy Markdown
Contributor Author

Nanashi-lab commented Oct 18, 2025

@vigoo all done.

@Nanashi-lab Nanashi-lab requested a review from vigoo October 18, 2025 15:37
@vigoo vigoo merged commit 933a860 into golemcloud:main Oct 18, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vigoo vigoo Awaiting requested review from vigoo

Assignees

No one assigned

Labels

🙋 Bounty claim

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add Cli support for listing and getting contents of worker files.

2 participants

@Nanashi-lab @vigoo