Skip to content

Add execution logs and SealGate notification icon - #40

Open
Miyamura80 wants to merge 2 commits into
mainfrom
feature/execution-logs-sealgate-icon
Open

Add execution logs and SealGate notification icon#40
Miyamura80 wants to merge 2 commits into
mainfrom
feature/execution-logs-sealgate-icon

Conversation

@Miyamura80

@Miyamura80 Miyamura80 commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Summary

Add a compact execution-log viewer for Mobile Bash and use the SealGate gateway mark consistently for Android notification identity.

Changes

  • Add a Logs action that opens the five most recent agent executions.
  • Let each execution row expand to show the full command, output, status, and duration.
  • Add Show more for the bounded 50-entry in-memory history.
  • Record running, completed, failed, and clipped-output Bash executions.
  • Replace the generic launcher bars with the SealGate gateway artwork.
  • Add a padded notification-safe SealGate glyph and adaptive monochrome layer.

Testing

  • Build passes (./gradlew assembleDebug)
  • Unit tests pass (./gradlew testDebugUnitTest)
  • Lint passes (./gradlew lintDebug)
  • Installed and exercised on Google Pixel 6 / Android 17
  • Verified the expanded notification displays the SealGate gateway icon

Related Issues

None


Summary by cubic

Adds a compact execution-log viewer for Mobile Bash and uses the SealGate gateway mark for Android notification identity.

  • Adds a Logs action in the main activity that opens the five most recent Bash executions.
  • Each execution row expands to show the full command, output, status, and duration.
  • A Show more button reveals the full 50-entry in-memory history.
  • Bash executions now record running, completed, failed, and clipped-output states.
  • Replaces the generic launcher bars with the SealGate gateway artwork and adds a padded notification-safe glyph and adaptive monochrome layer.

Written for commit 1e00f2a. Summary will update on new commits.

Review in cubic

@qodo-code-review

Copy link
Copy Markdown

ⓘ Qodo reviews are paused because the subscription is no longer active. Ask your workspace admin to reactivate the subscription to resume reviews. Manage billing

@cubic-dev-ai cubic-dev-ai 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.

1 issue found across 17 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="app/src/main/res/drawable/ic_notification_sealgate_safe.xml">

<violation number="1" location="app/src/main/res/drawable/ic_notification_sealgate_safe.xml:1">
P3: The new ic_notification_sealgate_safe.xml copies the full ~12KB path data of ic_stat_sealgate.xml (identical 2 paths / 12,321 chars each; the only delta is the added 0.62 scale group), and this PR removes the last usage of ic_stat_sealgate from TunnelService.kt, leaving it an unreferenced resource. Reuse one source of truth instead of duplicating the giant path twice: update ic_stat_sealgate.xml in place with the scale group and point the notification builder at it (or delete ic_stat_sealgate.xml), rather than adding a parallel 13KB drawable.</violation>
</file>

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread app/src/test/java/ai/sealgate/stdiod/ExecutionLogStoreTest.kt
Comment thread app/src/main/res/drawable/ic_notification_sealgate_safe.xml
Comment thread app/src/main/res/layout/activity_execution_log.xml Outdated
Comment thread app/src/main/res/drawable/ic_launcher_foreground.xml Outdated
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