Skip to content

feat: Add a user-facing report index with search and filtering #179 - #203

Open
lztmdrclht wants to merge 6 commits into
fmagent-project:mainfrom
lztmdrclht:fm-179
Open

feat: Add a user-facing report index with search and filtering #179#203
lztmdrclht wants to merge 6 commits into
fmagent-project:mainfrom
lztmdrclht:fm-179

Conversation

@lztmdrclht

Copy link
Copy Markdown
Contributor

Objective #179 :

After running FM-Agent, generate a report.html file to search for, view, and browse reports.

  • Each row/entry displays the title, status, file, function, and code location.
  • The search function supports text matching in titles, files, and functions.
  • Users can combine search with filters.
  • Users can sort the visible result set without losing their current filter settings.
  • Technical details for each entry can be expanded or collapsed.
  • The index is regenerated deterministically based on existing runtime artifacts, without invoking a large language model (LLM).
  • The documentation explains how to locate and open the results index.

Features to Be Implemented

It is unclear what the intended behavior is when opening a code file within an HTML file, so this will be addressed later.

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 05533edf04

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread report.py Outdated
Comment thread report.py Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d27f36c9d2

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread report.py Outdated
@lztmdrclht

Copy link
Copy Markdown
Contributor Author

@haoran-ding This PR needs to be reviewed.

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