Add migration epic document for forensic_analytics service extraction#98
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6516afb0de
ℹ️ 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".
| @@ -0,0 +1,201 @@ | |||
| Migration Epic: Auslagern der Analyse in den Dienst forensic_analytics | |||
There was a problem hiding this comment.
Rewrite epic document in English
The repository contract in AGENTS.md states that all repository documentation must be written in English, but this new epic starts in German and remains German throughout. This violates a mandatory project rule and makes the planning artifact harder to consume for the expected contributor audience and automation workflows, so the content should be translated before this commit is accepted.
Useful? React with 👍 / 👎.
| [1] Basics tutorial | Java | gRPC | ||
| https://grpc.io/docs/languages/java/basics/ | ||
| [2] Streaming messages between client and server services using gRPC remote procedure calls | ||
| https://openliberty.io/guides/grpc-intro.htmlMigration Epic: Auslagern der Analyse in den Dienst forensic_analytics |
There was a problem hiding this comment.
Remove duplicated epic content after references
The second full copy of the epic begins immediately after the [2] reference URL, so the citation line is malformed (...grpc-intro.htmlMigration Epic...) and the document now contains duplicate, diverging-maintenance content. This should be reduced to a single canonical block with a proper newline after the URL to avoid broken references and editing ambiguity.
Useful? React with 👍 / 👎.
Motivation
forensic_analyticsservice and that captures REST + gRPC interfaces, streaming behaviors, and migration scope.Description
epic.mdfile that contains the migration epic in full, including REST endpoints, a protobuf excerpt for gRPC services (AnalysisControlService,AnalysisResultService), the plugin workflow (remoteGenerateBtmRules), implementation task list, and open points (security, scaling, error handling).Testing
git status,git diff,git diff --cached, andnl -ba epic.md | sed -n '1,220p', and committed the new file asepic.md; all verification commands completed successfully.Codex Task