Add generated modal search and docs error reporting support#2356
Add generated modal search and docs error reporting support#2356niemyjski wants to merge 13 commits into
Conversation
niemyjski
force-pushed
the
feature/docs-pagefind-exceptionless-client
branch
from
01f630d to
3f3615e
Compare
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 01f630d7bb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1221593c96
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cf03ecf73a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
niemyjski
changed the title
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bdee5712d2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2344e5c9a5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 61a60c896e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1720ea9aa4
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 76bfd57089
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
niemyjski
force-pushed
the
feature/docs-pagefind-exceptionless-client
branch
from
1b673ee to
127a7d0
Compare
Use Deno Aspire resource for docs Polish docs search UI Use generated modal search for docs Address docs search review feedback Cancel docs search debounce on clear Fix docs search modal button styling Expose docs search keyboard shortcut Clarify docs search migration notes Preserve docs sidebar link color Align docs search nav on mobile Fix docs sidebar text color Restore docs sidebar link color Match docs sidebar colors to production Disable caching for local docs server Use existing nav styling for docs search trigger Adjust frontend package release age Reuse theme modal styles for docs search
niemyjski
force-pushed
the
feature/docs-pagefind-exceptionless-client
branch
from
127a7d0 to
f37513e
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f37513eca7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f88731a535
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
…-exceptionless-client
|
1 participant
Summary
deno task servethrough the CommunityToolkit Deno integration for normal development and omit it only from the existing--ci-e2epath/search/page and Pagefind task with a generated/search-index.jsonplus shared modal search opened from the header, shortcuts, or an input-shaped docs sidebar triggerEXCEPTIONLESS_SITE_*repository variables, initialize it before site/search JavaScript, and retain only theWebsitetag plus session and page context#78B001, content/search links use#578F01, sidebar section links use#78B001, and sidebar leaf links use#777Docs/Lumetags, reusable-workflow secret forwarding, the single-use bundle task, and the redundant explicit port argumentHistorical Color Notes
exceptionless/Websiteshows the docs TOC palette from Eric J. Smith's 2020-10-224543749"More styling" commit:.toc-list ais#78B001, while.toc-list a:only-childis#777.#78B001traces further back throughcontent/assets/css/theme.cssblame to Blake Niemyjski's 2015-09-08 WordPress theme CSS commitfe1a5252.#4f7600; those global overrides are removed while unrelated Lighthouse and search-dialog CSS remains intact.Verification
git diff --checkrefs/heads/mainat45cb8425, then merged that exact SHA as the second parent of merge commit6fb75f2b1; GitHub reports the PRMERGEABLEdotnet build Exceptionless.slnx --no-restore: 0 warnings, 0 errorsruby -e 'require "yaml"; YAML.load_file(".github/workflows/build.yaml"); YAML.load_file(".github/workflows/website.yml")'deno task checkdeno task verify: Lume build, postbuild search index generation, client bundle, and link verification passed with no broken local links, anchors, or assetsdeno task build: the configured output emits only API key, server URL, and version before/assets/js/site.js; the unconfigured output omits the client scriptPORT=7141 deno task serve: served/docs/; browser search forDockerreturned 11 resultsdotnet build src/Exceptionless.AppHost/Exceptionless.AppHost.csprojdenowith argstask serve; the--ci-e2emanifest contains no Docs resourceagent-browserchecked/,/why/,/tour/,/news/, and/docs/: headings resolve torgb(120, 176, 1)and content links resolve torgb(87, 143, 1)agent-browserchecked/docs/api/project-tokens/:API Usageisrgb(120, 176, 1),Client-Scoped Tokensisrgb(119, 119, 119), and content links arergb(87, 143, 1)agent-browserconfirmed the docs sidebar exposes one search trigger, clicking it opens an empty focused modal, and results appear after typing in the modal without an Enter handoffagent-browsersearched forimportfrom both/docs/and/, visually confirming JavaScript keyword, type, string, and query highlighting in generated code excerpts; the browser error log remained emptyagent-browserconfirmed stale search results no longer navigate: after a visibledockerresult, changing todefinitely-no-stale-result-matchand submitting stayed on the same docs page and showed zero results for the new queryagent-browserconfirmed modal focus remains inside the search dialog for both Tab and Shift+TabQueueEventMigrationsJobis searchable and the global fallback meta description is not indexed as page-specific description textBreaking Changes