Skip to content

feat(nav): give hermiq a Reports page, from the report it already had - #765

Merged
rubenvdlinde merged 1 commit into
developmentfrom
feat/app-chrome-reports
Sep 3, 2026
Merged

feat(nav): give hermiq a Reports page, from the report it already had#765
rubenvdlinde merged 1 commit into
developmentfrom
feat/app-chrome-reports

Conversation

@rubenvdlinde

Copy link
Copy Markdown
Contributor

ADR-114 Decision 3 puts Reports in every app. Nothing is invented here — AI oversight already existed, and it was an entry in the settings foldout.

That was the wrong home. The foldout is where configuration lives (ADR-044 Decision 3), and oversight is not configuration: it is a reading of what the agents did, which is what a report is.

⚠️ The route had to survive, and not only for deep links

dossiq declares /apps/hermiq/ai-oversight as an ADR-110 integrations entry. Retiring the menu entry while dropping the route would 404 from another app. The page is untouched and an e2e test navigates to it directly.

The deliberate asymmetry

Guardrail policy, the algorithm register and Compliance stay in the foldout — those genuinely are configuration. The e2e asserts it, so a later sweep that cards them fails a test rather than passing review.

Full chrome

Hermiq is one of only four apps that already had a Store, so its footer now carries all four declared items and the spec asserts the whole sequence: Documentation, Store, Reports, Features & roadmap. gate-107 reports zero warnings for this app — the first to reach that.

ChartBoxOutline registered in src/icons.js; an unregistered icon name renders no glyph at all.

Verified

gate-22, gate-53, gate-60, gate-102, gate-104 and gate-107 clean; prettier clean; all five e2e tests collect.

🤖 Generated with Claude Code

ADR-114 Decision 3 puts Reports in every app. Nothing is invented here: AI
oversight already existed, and it was an entry in the SETTINGS foldout.

That was the wrong home. The foldout is where configuration lives (ADR-044
Decision 3), and oversight is not configuration — it is a reading of what the
agents did, which is what a report is. It becomes a card on /reports.

⚠️ THE ROUTE HAD TO SURVIVE, AND NOT ONLY FOR DEEP LINKS. dossiq declares
/apps/hermiq/ai-oversight as an ADR-110 integrations entry, so retiring the
menu entry while dropping the route would 404 from ANOTHER app. The page is
untouched, and an e2e test navigates to it directly.

Guardrail policy, the algorithm register and Compliance stay in the foldout.
Those ARE configuration, and the e2e asserts that asymmetry so a later sweep
fails a test rather than passing review.

Hermiq is one of only four apps that already had a Store, so its footer now
carries all four declared chrome items and the spec asserts the full sequence
Documentation, Store, Reports, Features & roadmap.

ChartBoxOutline registered in src/icons.js. An unregistered icon name renders
no glyph at all — not a fallback, not a console error.

Verified: gate-22, gate-53, gate-60, gate-102, gate-104 and gate-107 clean
(gate-107 now reports zero warnings for this app, since it has both Store and
Reports); prettier clean; all five e2e tests collect.
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/hermiq @ 1a83e6f

Check PHP Vue Security License Tests
lint
phpcs
phpmd
psalm
phpstan
phpmetrics
eslint
stylelint
build
check-specs
check-manifest
test-l10n
format
check-l10n-js
check-schema-l10n
composer ✅ 123/123
npm ✅ 730/730
app:check-code ⏭️
info.xml
REUSE
PHPUnit
Newman
Playwright ⏭️ deferred — runs on the promotion into beta/main, not on a pull request into development
Hydra gates

Quality workflow — 2026-09-03 17:17 UTC

Download the full PDF report from the workflow artifacts.

@rubenvdlinde
rubenvdlinde merged commit ed5d9ca into development Sep 3, 2026
50 checks passed
@rubenvdlinde
rubenvdlinde deleted the feat/app-chrome-reports branch September 3, 2026 17:23
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.

1 participant