Skip to content

Conversation

@reynaldichernando
Copy link
Contributor

@reynaldichernando reynaldichernando commented Oct 13, 2025

knowing that non-interactive mode is definitely possible (#23 (comment)) and not blocked by auth api

i'm adding puter site deploy as the first command to properly have non-interactive mode

usage

puter site deploy [local_dir] [subdomain]

@reynaldichernando

This comment was marked as outdated.

@reynaldichernando
Copy link
Contributor Author

reynaldichernando commented Oct 13, 2025

so right now it just run deploy by doing startShell('site:deploy...')

although ideally we don't do this, and just run the internal deploy function

this means we gotta figure out how to properly init ProfileModule and other modules when running non interactive

another thing we gotta do is to cleanup the logs a little bit

@reynaldichernando

This comment was marked as outdated.

@reynaldichernando
Copy link
Contributor Author

technically the command works right now, but the doing startshell is a bit terrible

@reynaldichernando reynaldichernando marked this pull request as draft October 13, 2025 08:55
@bitsnaps
Copy link
Collaborator

bitsnaps commented Oct 13, 2025

Amazing work @reynaldichernando , many thanks!
I'll look into that in more detail, and consider all these improvements for the v2.

@reynaldichernando
Copy link
Contributor Author

image added the prompt for local dir and subdomain so now both local dir and subdomain are optional image

@reynaldichernando
Copy link
Contributor Author

btw for now i won't touch the ProfileModule yet, i'll just have it at another PR, which also tackle the #23
i'll just cleanup the logs (some are still copy pasted from app) and then mark this ready to review

@reynaldichernando
Copy link
Contributor Author

the logs cleanup

  • remove duplicate logs (since we have both in deploy and createSite)
  • fix createSite log app -> site
  • make random name have more generic log

@reynaldichernando reynaldichernando marked this pull request as ready for review October 14, 2025 10:59
@reynaldichernando
Copy link
Contributor Author

hi @bitsnaps , this is ready to review 🫡

@bitsnaps bitsnaps merged commit 0280b5a into HeyPuter:master Oct 18, 2025
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.

2 participants