MR-17 Curious Reader component logic to deal with "minimum" received sub-app payload data

[Rajesh1041]

Changes

• app/src/main/java/org/curiouslearning/container/WebApp.java
• app/src/main/java/org/curiouslearning/container/core/subapp/handler/AppEventPayloadHandler.java
• app/src/main/java/org/curiouslearning/container/core/subapp/handler/DefaultAppEventPayloadHandler.java
• app/src/main/java/org/curiouslearning/container/core/subapp/payload/AppEventPayload.java
• app/src/main/java/org/curiouslearning/container/core/subapp/validation/AppEventPayloadValidator.java
• app/src/main/java/org/curiouslearning/container/core/subapp/validation/ValidationResult.java

How to test

• Trigger via JS with payload

Ref: MR-17

Summary by CodeRabbit

• New Features
  • Receive and process JSON-formatted event payloads from web interfaces.
  • Validate incoming payloads rigorously, returning clear failure reasons for malformed or incomplete data.
  • Route validated events to a dedicated handler and emit structured logs with event metadata for monitoring.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds JSON payload processing for app events: new payload model, validator and validation result, handler interface and default implementation, and a WebAppInterface.logMessage(String) method that parses, validates, and delegates payloads with error handling.

Changes

Cohort / File(s)	Summary
Payload Model app/src/main/java/org/curiouslearning/container/core/subapp/payload/AppEventPayload.java	New POJO with public fields: cr_user_id, app_id, collection, data, options, timestamp for Gson deserialization.
Validation app/src/main/java/org/curiouslearning/container/core/subapp/validation/ValidationResult.java, app/src/main/java/org/curiouslearning/container/core/subapp/validation/AppEventPayloadValidator.java	ValidationResult models success/failure with message; AppEventPayloadValidator enforces required fields and validates options values (allowed: "add", "replace").
Handler app/src/main/java/org/curiouslearning/container/core/subapp/handler/AppEventPayloadHandler.java, app/src/main/java/org/curiouslearning/container/core/subapp/handler/DefaultAppEventPayloadHandler.java	New handler interface AppEventPayloadHandler with handle(AppEventPayload). Default implementation logs payload details (app_id, collection, timestamp).
Integration / WebApp app/src/main/java/org/curiouslearning/container/WebApp.java	WebAppInterface gains logMessage(String payloadJson) plus private fields: Gson gson, AppEventPayloadValidator validator, AppEventPayloadHandler handler. Method parses JSON, validates payload, delegates to handler, and logs errors (including JSON syntax exceptions).

Sequence Diagram

sequenceDiagram
    participant Client
    participant WebApp as WebAppInterface
    participant Gson
    participant Validator as AppEventPayloadValidator
    participant Handler as DefaultAppEventPayloadHandler

    Client->>WebApp: logMessage(payloadJson)
    WebApp->>WebApp: check non-empty payload
    alt payload empty
        WebApp-->>Client: log error / return
    else payload present
        WebApp->>Gson: fromJson(payloadJson)
        Gson-->>WebApp: AppEventPayload
        WebApp->>Validator: validate(payload)
        Validator-->>WebApp: ValidationResult
        alt valid
            WebApp->>Handler: handle(payload)
            Handler-->>Handler: Log app_id, collection, timestamp
        else invalid
            WebApp-->>WebApp: log validation error
        end
    end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Poem

🐰 I hopped into JSON, neat and bright,
Parsed each field by morning light.
I checked the keys, then passed the call,
Logged the event for one and all.
Hooray—payloads handled, springtime delight! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Tests & Lint & Coverage	⚠️ Warning	PR adds six new Java classes with business logic but includes no corresponding unit tests, test coverage configuration, or linting setup.	Add unit tests for all new classes, configure Jacoco for coverage, set up Android linting rules, and add CI workflows to enforce testing requirements.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The pull request title clearly and directly describes the main change: implementing logic to handle minimum sub-app payload data in the Curious Reader component, and includes the JIRA key MR-17 as required.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 2

🤖 Fix all issues with AI agents

In `@app/src/main/java/org/curiouslearning/container/WebApp.java`:
- Around line 19-20: The file WebApp.java contains duplicate imports for
com.google.gson.Gson and com.google.gson.JsonSyntaxException; remove the
redundant import statements so each type is imported only once (locate the
duplicate import entries for Gson and JsonSyntaxException in WebApp.java and
delete the extra occurrences), ensuring the remaining imports are clean and the
class WebApp compiles without duplicate import warnings.
- Line 24: Remove the duplicate import of AppEventPayload in WebApp.java: locate
the redundant import statement for
org.curiouslearning.container.core.subapp.payload.AppEventPayload (it appears
twice) and delete one of them so the class is imported only once; ensure no
other needed imports are removed and that WebApp compiles with the single import
remaining.

🧹 Nitpick comments (1)

app/src/main/java/org/curiouslearning/container/core/subapp/validation/AppEventPayloadValidator.java (1)

31-33: Consider validating timestamp format.

The validator checks that timestamp is present but not that it conforms to a valid format (e.g., ISO 8601). If payloads with malformed timestamps could cause issues downstream, consider adding format validation here.