datatug / datatug-cli / 26883351596

62%

Build Type

push

github

Committed by trakhimenok

Commit Message

fix(cli): wire scan/projects-add flags and stop scan from panicking

Closes #136, closes #137.

After the migration to urfave/cli/v3 the command structs still carried
jessevdk/go-flags struct tags, but their flags were never registered and the
action structs were never populated — so `scan` and `projects add` could not
receive input, and `scan` used panic() on unimplemented paths.

scan (#136):
- register all flags (-p/-d project selection, -D/-s/--port/-U/-P/--db/
  --dbmodel/--env) and populate scanDbCommand from the parsed command
- extract connection-string construction into scanDbCommand.connectionParams(),
  which returns an error (no more panic) for the not-yet-implemented paths
  (sqlite3, and env-derived host)

projects add (#137):
- register -p/--project and -d/--directory and populate the command
- replace the broken local saveConfig (created a literal "~/.datatug.yaml")
  with dtconfig.SaveSettings; point cmd_demo.go at the same helper

Also: resolve a locally-added project (stored as Path) by -p/--project and in
the project store, not only Origin-based repos.

Tests: connectionParams (sqlite3/no-host error, sqlserver ok), flag
registration for both commands, and getProjPathsByID Path/Origin fallback.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

30 of 62 new or added lines in 5 files covered. (48.39%)

4 existing lines in 3 files now uncovered.

5510 of 14290 relevant lines covered (38.56%)

0.45 hits per line