Skip to content

Add a command line version#18

Draft
sbutler wants to merge 4 commits into
tlsref:masterfrom
sbutler:master
Draft

Add a command line version#18
sbutler wants to merge 4 commits into
tlsref:masterfrom
sbutler:master

Conversation

@sbutler

@sbutler sbutler commented Jul 6, 2026

Copy link
Copy Markdown

I would like to use this project to generate configurations for use in Ansible. Having a command line version I can run with facts discovered from Ansible would be very useful. I used Claude Code to quickly generate a CLI command and add it to the project, then reviewed the code. Most of it is copied from src/js/state.js with adjustments for running under NodeJS.

@gstrauss

gstrauss commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

I used Claude Code to quickly generate ...

"a lot of work for somebody else." There, FTFY.

Most of it is copied from src/js/state.js

Fantastic failure of a junior coder.
Please justify why the code needed to be copied rather than smaller adjustments made for reuse.

@gstrauss gstrauss marked this pull request as draft July 6, 2026 17:23
@sbutler

sbutler commented Jul 6, 2026

Copy link
Copy Markdown
Author

I thought it would be more likely to be acceptable if it did not modify the existing code paths, and the state file currently had a lot of browser specific items. However, if you'd like a refactor to share code I have made those changes.

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