Skip to content

QUA-1756: Add Operations Runs documentation and document Auto-Resolve#1136

Open
RafaelOsiro wants to merge 5 commits into
mainfrom
qua-1756-auto-resolve-anomalies
Open

QUA-1756: Add Operations Runs documentation and document Auto-Resolve#1136
RafaelOsiro wants to merge 5 commits into
mainfrom
qua-1756-auto-resolve-anomalies

Conversation

@RafaelOsiro

@RafaelOsiro RafaelOsiro commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Overview

Adds the new Operations → Runs documentation section and rounds out the QUA-1756 Auto-Resolve coverage that started in the Scan Operation rewrite. Documents Run lifecycle, available actions, permissions, per-state walkthroughs for Scan Runs, and refreshes the Operations overview hub.

Key Changes

  • Operations → Runs section: Getting Started, Deep Dive (Introduction, Lifecycle, Available Actions, Permissions), By Operation Type (Scan in 5 states: Success, Success with Warning, Failure, Aborted, Running), API, and FAQ.
  • FE-validated state icons and colors: every Run state icon and Timeline marker matches the actual frontend (Success → mdi-check-circle, Failure → mdi-close-circle, Aborted → mdi-alert-circle warning, Running/Queued → mdi-loading).
  • Auto-Resolved indicator (composite icon): new .qua-icon-auto-resolved CSS utility renders the filled triangle + check overlay used on Scan Runs.
  • Operations overview refresh: card grid uses the FE-aligned operation icons; H1 icons added in brick on Sync, Profile, Scan, External Scan, Export, Materialize, and Promote.
  • Legacy cleanup: removed scan/how-tos/interpret-results.md (replaced by per-state pages under runs/by-types/scan/); ran the Scan run-state screenshots through the migration to docs/assets/operations/runs/by-types/scan/.
  • Nav reorder: Scan submenu now reads Troubleshooting → API → FAQ; Scan Getting Started Reference cards reordered to match.
  • First-class Container sub-tabs: Partitions / Checks / Source Records now rendered as H5 sub-sections with each screenshot inline (replaces the previous monolithic table + dumped screenshots).

Pages to Test

Operations Overview

Runs — Getting Started + Deep Dive

Runs — By Operation Type (Scan)

Runs — API + FAQ

Cross-cutting

  • http://localhost:8000/operations/scan/getting-started/ — Reference grid order is Troubleshooting → API → FAQ.
  • Light and dark mode: confirm the qua-icon-auto-resolved composite (filled triangle + white/dark check overlay) renders inline at body-text size and inverts contrast correctly when switching themes.

@RafaelOsiro RafaelOsiro added the documentation Improvements or additions to documentation label Jun 6, 2026
@RafaelOsiro RafaelOsiro self-assigned this Jun 6, 2026
@RafaelOsiro RafaelOsiro marked this pull request as draft June 6, 2026 03:15
Adds the Operations → Runs documentation: Getting Started, Deep Dive
(Introduction, Lifecycle, Available Actions, Permissions), By Operation
Type (Scan in 5 states), API, and FAQ. Migrates Scan run-state screenshots
to the new layout, refreshes the Operations overview, aligns sibling
operation H1 icons, removes the legacy interpret-results page, and adds
the qua-icon-auto-resolved CSS utility for the composite Auto-Resolved
indicator. Also reorders the Scan submenu to Troubleshooting → API → FAQ.
@RafaelOsiro RafaelOsiro changed the title QUA-1756: Restructure Scan Operation docs and document Auto-Resolve QUA-1756: Add Operations Runs documentation and document Auto-Resolve Jun 18, 2026
@RafaelOsiro RafaelOsiro marked this pull request as ready for review June 18, 2026 01:03

@ets ets left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed for obvious errors. The restructure is solid — nav targets, redirect map, image paths, include-markdown directives, and admonitions/code fences all check out. One concrete issue worth fixing before merge:

Two broken internal links to the deleted scan/scan.md page (with a dead anchor):

This PR deletes docs/operations/scan/scan.md and moves the Unloadable Container Error section to docs/operations/scan/troubleshooting.md (#unloadable-container-error). Two files still point at the old path + anchor:

  • docs/operations/profile/profile.md:753[Unloadable Container Error](../scan/scan.md#unloadable-container-error)
  • docs/operations/sync/sync.md:43[Unloadable Container Error](../scan/scan.md#unloadable-container-error)

The mkdocs.yml redirect (operations/scan/scan.mdgetting-started.md) redirects the page but drops the #unloadable-container-error anchor, which doesn't exist on getting-started.md — so these land on the wrong page with a dead anchor. The correct target is ../scan/troubleshooting.md#unloadable-container-error (which this PR's own select-tables.md:29 already uses correctly).

Everything else looks good.


Generated by Claude Code

…shooting.md

The Profile and Sync pages still linked to the deleted scan/scan.md
page with the #unloadable-container-error anchor. The mkdocs redirect
on the parent page drops the anchor, so the old links landed on
scan/getting-started/ with a dead anchor. Repoint at the actual home
of the section: scan/troubleshooting.md#unloadable-container-error.
@RafaelOsiro

Copy link
Copy Markdown
Contributor Author

Addressed in 322e70c0. Both broken ../scan/scan.md#unloadable-container-error references now point at ../scan/troubleshooting.md#unloadable-container-error:

  • docs/operations/profile/profile.md:753
  • docs/operations/sync/sync.md:43

@greptileai

@greptile-apps

greptile-apps Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds the complete Operations → Runs documentation section (Getting Started, Deep Dive, five Scan run-state walkthroughs, API, and FAQ), refreshes the Operations overview hub with FE-aligned icons, and introduces the .qua-icon-auto-resolved CSS utility. All three issues flagged in the previous review round are confirmed resolved in 6d6b76d5.

  • New Runs section: Getting Started, Deep Dive (Introduction, Lifecycle, Available Actions, Permissions), By Operation Type (Scan in 5 states), API, and FAQ are all added with consistent structure, correct cross-links, and FE-validated icon/color assignments throughout.
  • FE color fixes addressed: Aborted Timeline markers now use var(--q-warning) across all five Scan state pages; failure.md adds a Resume callout; --q-info is intentionally identical in light/dark blocks to mirror the frontend palette sync.
  • One badge-color mismatch remaining: running.md describes the Running status badge as "orange" in four places, but the frontend badge component uses bg-tertiary/10 text-tertiary (#5B86AA, blue/steel) — confirmed against src/components/operation/badge/index.vue.

Confidence Score: 4/5

Safe to merge after correcting the Running badge color from "orange" to blue/tertiary in running.md.

The Running badge is documented as "orange" across four places in running.md, but the frontend badge component styles it as blue/tertiary (#5B86AA). This directly contradicts the PR's FE-validated-colors goal and would cause confusion for readers trying to identify the Running state in the UI. Everything else in the new documentation section — icon assignments, timeline colors, permission matrices, nav order, and API reference — is accurate and consistent.

docs/operations/runs/by-types/scan/running.md — four occurrences of "orange" for the Running badge need to be corrected to blue/tertiary.

Important Files Changed

Filename Overview
docs/operations/runs/by-types/scan/running.md New Scan Running state page — badge color documented as "orange" but FE badge component uses blue/tertiary (#5B86AA) in 4 places.
docs/operations/runs/by-types/scan/failure.md New Scan Failure state page — previously flagged issues resolved: Timeline Aborted color now uses var(--q-warning), Resume callout added.
docs/operations/runs/by-types/scan/aborted.md New Scan Aborted state page — Timeline icon legend correctly uses var(--q-warning) for Aborted. Resume documented in both row Settings table and intro.
docs/operations/runs/by-types/scan/success-with-warning.md New Scan Success with Warning page — Timeline Aborted marker correctly uses var(--q-warning). Auto-Resolved composite icon documented correctly.
docs/operations/runs/by-types/scan/success.md New Scan Success state page — Timeline legend colors are consistent with introduction.md and FE-validated spec.
docs/operations/runs/deep-dive/introduction.md New introduction page — States at a glance table correctly uses FE-validated icon/color assignments for all five states.
docs/operations/runs/deep-dive/actions.md New Available Actions page — support matrix accurately lists which operation types support Resume vs Rerun, with correct permission gating.
docs/operations/runs/deep-dive/permissions.md New Permissions page — five-row team permission matrix is accurate; Admin bypass callout is correct; cross-links to promote/scan permissions are valid.
docs/operations/runs/api.md New Runs API page — endpoint URLs and operation_type enum values are correct; catalog clarification for Sync is in place.
docs/operations/runs/faq.md New Runs FAQ — em-dash removed from Warning indicator description; Admin user role (not workspace role) is correct.
docs/stylesheets/extra.css Adds .qua-icon-auto-resolved composite CSS utility; --q-info kept identical in light/dark by design to mirror frontend palette sync.
docs/operations/overview.md New Operations overview hub — 8 operation cards with FE-aligned outline icons; card for Runs added as first entry.
mkdocs.yml Nav updated — Runs section added with correct hierarchy; Scan submenu reordered to Troubleshooting → API → FAQ as specified.
docs/operations/runs/getting-started.md New Runs Getting Started page — section grids for Deep Dive, By Operation Type, API and FAQ link correctly to all subpages.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    Q([Queued\n🔵 blue/info]) -->|worker picks up| R([Running\n🔵 blue/tertiary])
    R -->|completes cleanly| S([Success\n🟢 positive])
    R -->|completes with log entries| SW([Success + Warning indicator\n🟢 positive + ⚠️ warning])
    R -->|fatal error| F([Failure\n🔴 negative])
    R -->|user/system stop| A([Aborted\n⚠️ warning])
    F -->|Resume – profile/scan/promote| R
    A -->|Resume – profile/scan/promote| R
    S -->|Rerun| Q
    SW -->|Rerun| Q
    F -->|Rerun| Q
    A -->|Rerun| Q
    S -->|Delete| DEL([Record removed])
    SW -->|Delete| DEL
    F -->|Delete| DEL
    A -->|Delete| DEL
    style Q fill:#2163D0,color:#fff
    style R fill:#5B86AA,color:#fff
    style S fill:#21BA45,color:#fff
    style SW fill:#21BA45,color:#fff
    style F fill:#E91E63,color:#fff
    style A fill:#ffbb33,color:#000
    style DEL fill:#9e9e9e,color:#fff
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    Q([Queued\n🔵 blue/info]) -->|worker picks up| R([Running\n🔵 blue/tertiary])
    R -->|completes cleanly| S([Success\n🟢 positive])
    R -->|completes with log entries| SW([Success + Warning indicator\n🟢 positive + ⚠️ warning])
    R -->|fatal error| F([Failure\n🔴 negative])
    R -->|user/system stop| A([Aborted\n⚠️ warning])
    F -->|Resume – profile/scan/promote| R
    A -->|Resume – profile/scan/promote| R
    S -->|Rerun| Q
    SW -->|Rerun| Q
    F -->|Rerun| Q
    A -->|Rerun| Q
    S -->|Delete| DEL([Record removed])
    SW -->|Delete| DEL
    F -->|Delete| DEL
    A -->|Delete| DEL
    style Q fill:#2163D0,color:#fff
    style R fill:#5B86AA,color:#fff
    style S fill:#21BA45,color:#fff
    style SW fill:#21BA45,color:#fff
    style F fill:#E91E63,color:#fff
    style A fill:#ffbb33,color:#000
    style DEL fill:#9e9e9e,color:#fff
Loading

Reviews (2): Last reviewed commit: "docs(runs): align Aborted Timeline color..." | Re-trigger Greptile

Comment thread docs/operations/runs/by-types/scan/failure.md Outdated
Comment thread docs/operations/runs/by-types/scan/failure.md Outdated
Comment thread docs/stylesheets/extra.css
… Resume on Scan Failure

- Timeline icon legend on success/success-with-warning/failure/aborted/
  running pages now colors the Operation Aborted terminal marker with
  --q-warning (amber) to match the 'States at a glance' table on the
  Runs Introduction page. Previously it was --q-negative (red), which
  conflicted with the Aborted state badge color throughout the section.
- Scan Failure: the row description and a follow-up note now mention
  that a Resume button appears alongside Rerun/Delete on recoverable
  failures. The Settings table keeps the numbered rows aligned with the
  screenshot annotations (Results/Rerun/Delete), with a callout above
  pointing to the Available Actions reference for the full rules.
@RafaelOsiro

Copy link
Copy Markdown
Contributor Author

Addressed in 6d6b76d5:

  • Aborted Timeline colorvar(--q-negative)var(--q-warning) on the Timeline icon legend across all 5 Scan run-state pages (success, success-with-warning, failure, aborted, running). Now matches the "States at a glance" color in runs/deep-dive/introduction.md and the FE-validated assignment in the PR description.
  • Resume on Scan Failure — the row description on failure.md now lists "Results, Resume (when supported), Rerun, Delete". The Settings table keeps its numbered rows aligned with the screenshot annotations (10/Results, 11/Rerun, 12/Delete), with a follow-up !!! note "Resume on recoverable failures" callout that points readers at the Available Actions reference for the full rules.
  • --q-info light/dark parity — keeping #2163D0 in both blocks by design. The userguide's "Frontend Palette Sync" section mirrors src/css/quasar.variables.scss on the frontend, where $info is defined once and reused in both light and dark modes. Reply posted on the thread.

All three threads resolved.

@greptileai

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants