Some updates to the `tctl access-review` command and skill based on some
feedback from Roman. The changes here include:
1) Using `Granted By` instead of `Grantor` to reduce confusion around
the noun (other columns, fields changed accordingly)
2) Defaulting the activitity window to 24h (instead unset by default) so
that if IAC is available the access information is returned by
default
3) Splitting the IAC error/warning into it's own JSON/YAML field and
giving it a dedicated location in the text output (this is paired
with an Access Graph PR that returns a `Not Configured` message when
IAC is not enabled on the cluster (with links to docs).
* feat: text output, geo, and `--skill` support
Layer three orthogonal additions on top of the core command:
- Text rendering: period header, matches summary, wrapped facet
panel, events table. `--all-facets` opts out of the top-5
truncation.
- Geo filter (`--latitude`/`--longitude`/`--radius`, all three
required together). Geo applies to events only; the stats endpoint
has no geo params, so facet counts and `total` cover the
window+query without geo. Text output prints a Note when geo is
active.
- `--skill` prints an embedded Markdown skill for LLM agents and
skips the backend entirely.
* fix: update skill to follow RFD 0037e
* fix: small fixes reported in PR review
* fix: small suggestion about code comment
* feat: iteration based on notion/customer request
* feat: add sanitization to text rendering
* fix: small fix to the access request review
* fix: typos and PR suggestions
* fix: help message indentation
* First Draft of teleport-session-review skill
Framework based on ACL skill
* Adds Examples and Playbook for how to use Session Search
* Add a preflight check
* Feedback from Docs
* Extra Testing on a Teleport Cloud Tenant
* Docs to Skill
* Remove local dev for now
* update locate commands to include $TSH
* Update docs/pages/identity-security/session-summaries/session-search.mdx
Co-authored-by: Paul Gottschling <[email protected]>
* Remove Filter caveats
These have been fixed
---------
Co-authored-by: Paul Gottschling <[email protected]>
Ghassan Gedeon Achi