feat: add config file support, rewrite README, automate releases#1
feat: add config file support, rewrite README, automate releases#1MasonEgger wants to merge 2 commits intomainfrom
Conversation
Add TOML config file support that reads from the [ts-net] section of the Temporal CLI config file (~/.config/temporalio/temporal.toml). This shares the same config file the CLI uses, with standard flag > env var > config file > defaults precedence. Rewrite README with proper structure: prerequisites, install instructions with /usr/local/bin guidance, usage, flag reference, configuration docs, a "How to use" section showing CLI and SDK environment configuration for connecting from other machines, and a "How it works" overview. Other changes: - Add VERSION file and tag-release GitHub Action to automate releases - Add install.sh for curl-pipe-to-sh installation - Bump tailscale.com from v1.82.5 to v1.96.5 - Set goreleaser project_name to temporal-ts-net for correct archive names - Add CONTRIBUTORS.md and CLAUDE.md - Add --config flag and flag-tracking to ParseExtensionArgs - Add config_test.go with 18 test cases covering loading, merging, path resolution, and three-way precedence - Add commit-msg.md to .gitignore
| permissions: | ||
| contents: write | ||
|
|
||
| jobs: |
There was a problem hiding this comment.
I just use normal github released. not super opinionated in this project though
There was a problem hiding this comment.
This is a way of having github do the go release as part of CI merge to main
| @@ -0,0 +1,101 @@ | |||
| # Session Summary: Config File Support, README Rewrite, Release Automation | |||
There was a problem hiding this comment.
This is how I've been doing my AI workflows, but I can delete them.
| INSTALL_DIR="/usr/local/bin" | ||
|
|
||
| # Detect OS | ||
| OS=$(uname -s) | ||
| case "$OS" in | ||
| Darwin) OS="Darwin" ;; | ||
| Linux) OS="Linux" ;; | ||
| *) | ||
| echo "Unsupported OS: $OS" | ||
| exit 1 | ||
| ;; | ||
| esac |
There was a problem hiding this comment.
Does windows even use this method of installation? I guess I could change the message to say "installer unsupported, use manual install instructions."
| @@ -0,0 +1 @@ | |||
| 0.0.3 | |||
There was a problem hiding this comment.
not familiar with having a version file
There was a problem hiding this comment.
This is a lazy version of having a GitHub action see this changes, then doing all the go-releaser stuff itself in CI when new version is merged to main
| ```toml | ||
| [ts-net] | ||
| tailscale-hostname = "my-temporal" | ||
| tailscale-authkey = "tskey-auth-..." | ||
| tailscale-state-dir = "/path/to/state" | ||
| max-connections = 500 | ||
| connection-rate-limit = 50 | ||
| dial-timeout = "15s" | ||
| idle-timeout = "10m" | ||
| ``` |
There was a problem hiding this comment.
nice!
not familiar yet with the config toml but seems inline with cloud-cli
There was a problem hiding this comment.
Ya i had it compare to that
Add TOML config file support that reads from the [ts-net] section of the Temporal CLI config file (~/.config/temporalio/temporal.toml). This shares the same config file the CLI uses, with standard flag > env var > config file > defaults precedence.
Rewrite README with proper structure: prerequisites, install instructions with /usr/local/bin guidance, usage, flag reference, configuration docs, a "How to use" section showing CLI and SDK environment configuration for connecting from other machines, and a "How it works" overview.
Other changes: