Skip to content

Named profiles + nightly toolchain + dep updates - #16

Merged
ankarhem merged 10 commits into
mainfrom
feat/profiles
Sep 15, 2026
Merged

ankarhem merged 10 commits into
mainfrom
feat/profiles

Conversation

@ankarhem

@ankarhem ankarhem commented Sep 15, 2026

Copy link
Copy Markdown
Member

Named profiles for multiple Graylog instances, plus supporting changes.

Profiles (3fa0d24 + follow-ups)

  • Config is now { profiles, active_profile, updater }; legacy [graylog] migrates in-memory to profiles.default without re-auth
  • Global --profile (GRAYLOG_PROFILE), profiles list/show/use/rename/delete; ping/auth report profile + available_profiles
  • Profile errors as visible ValidationError; token-free summaries; per-profile fields-<profile> cache keys
  • 282 lib + 18 integration tests green, clippy -D warnings clean

Also in this PR

  • Deps to latest semver-compatible (Cargo.lock)
  • Flake: crane + single nightly fenix toolchain, split deps, lean ci shell; clippy/test/deny CI jobs use .#ci
  • AGENTS.md knowledge base; skill docs for profiles with per-OS config paths

Fixes CI issues on this branch before merge; release to follow with a fresh tag after merge.


Devin Review

Init-deep inline path (S~345KB, N_quick=1). Root only, no subdirs warranted.
- nixpkgs stays nixos-unstable, add crane, one stable toolchain everywhere
- buildDepsOnly -> cargoArtifacts split, cleanCargoSource, strictDeps
- devShell default (bacon/cargo-deny/cargo-edit, drop cargo-watch/wizard/nextest/udeps) + lean ci shell
- clippy/test/deny CI jobs use .#ci; treefmt adds prettier
- Config is now {profiles, active_profile, updater}; legacy [graylog]
  migrates in-memory to profiles.default without re-auth
- Global --profile (GRAYLOG_PROFILE), profiles list/use/show/delete,
  ping and auth report profile + available_profiles
- ValidationError for profile misuse, token-free summaries,
  per-profile fields-{profile} cache keys
- New profiles rename command (validates both names, rejects
  unknown source and existing target, follows active selection)
- skills/graylog-cli: add Profiles section (storage layout,
  management commands, --profile/GRAYLOG_PROFILE, ping fields)
Copilot AI lite review requested due to automatic review settings September 15, 2026 16:03
devin-ai-integration[bot]

This comment was marked as resolved.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

- nix build: set doCheck=false on crane builds; the ping
  integration test needs loopback networking, which the nix
  sandbox blocks (suite runs with network in the CI test job)
- cargo-deny: ignore RUSTSEC-2026-0173 (proc-macro-error2,
  unmaintained transitive via latest tabled_derive, compile-time only)
- profiles: invalidate per-profile fields cache on re-auth so a
  replaced profile cannot serve the previous server field list
- profiles: reject case-insensitive duplicate names on auth and
  rename (cache files live on case-insensitive filesystems)
- treefmt: prettier realignment of skill docs table
Copilot AI review requested due to automatic review settings September 15, 2026 16:13

This comment was marked as resolved.

- nix builds: disable checks for the Windows cross package too;
  integration tests need loopback, unavailable in the sandbox
- config load: reject profile keys that fail name validation so
  hand-edited keys cannot reach cache file paths
- profiles list/show: validate a --profile override instead of
  reporting a phantom active profile
- tests: set APPDATA in the harness so Windows resolves the temp
  config dir; AGENTS.md per-profile cache key doc
Copilot AI review requested due to automatic review settings September 15, 2026 16:33

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@ankarhem
ankarhem merged commit bc3bba0 into main Sep 15, 2026
8 checks passed
@ankarhem
ankarhem deleted the feat/profiles branch September 15, 2026 16:47
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