Skip to content

build(deps): bump aw-notify from f63810d to 969352f#1336

Open
dependabot[bot] wants to merge 1 commit into
masterfrom
dependabot/submodules/aw-notify-969352f
Open

build(deps): bump aw-notify from f63810d to 969352f#1336
dependabot[bot] wants to merge 1 commit into
masterfrom
dependabot/submodules/aw-notify-969352f

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Jul 1, 2026

Copy link
Copy Markdown
Contributor

Bumps aw-notify from f63810d to 969352f.

Commits
  • 969352f add MPL-2.0 LICENSE file
  • 1426676 feat: add aw-notify-client crate and harden the /notify endpoint
  • 8c8cf6f feat: add minimal delay for displaying notifications and rate limits
  • 352e01b feat: notification support for other modules
  • See full diff in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [aw-notify](https://github.com/ActivityWatch/aw-notify-rs) from `f63810d` to `969352f`.
- [Release notes](https://github.com/ActivityWatch/aw-notify-rs/releases)
- [Commits](ActivityWatch/aw-notify-rs@f63810d...969352f)

---
updated-dependencies:
- dependency-name: aw-notify
  dependency-version: 969352f65b89fb60eb65df06a58adb2a8db6b7f7
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file submodules Pull requests that update Submodules code labels Jul 1, 2026
@greptile-apps

greptile-apps Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR bumps the aw-notify git submodule from f63810d to 969352f, incorporating four upstream commits from the aw-notify-rs repository.

  • Adds an MPL-2.0 LICENSE file to the submodule.
  • Introduces a new aw-notify-client crate, hardens the /notify HTTP endpoint, and adds rate-limiting plus a minimal display delay for notifications.
  • Extends notification support to additional modules beyond the original scope.

Confidence Score: 5/5

Safe to merge — this is a submodule pointer update that pulls in a license file, rate-limiting logic, and a new client crate with no changes to the parent repository's own code.

The only change is a single-line submodule commit hash update. The upstream commits add a LICENSE file, harden the notification endpoint with rate limiting, and extend notification support — all additive, low-risk changes. There are no modifications to parent-repo code, build scripts, or configuration.

No files require special attention.

Important Files Changed

Filename Overview
aw-notify Git submodule pointer bumped from f63810d to 969352f, pulling in license addition, notification hardening, rate limiting, and multi-module notification support

Sequence Diagram

%%{init: {'theme': 'neutral'}}%%
sequenceDiagram
    participant Module as Other Modules
    participant Client as aw-notify-client
    participant Server as aw-notify /notify
    participant NotifySys as OS Notification

    Module->>Client: send notification request
    Client->>Server: POST /notify (hardened endpoint)
    Server->>Server: check rate limit / min delay
    alt within rate limit
        Server->>NotifySys: display notification
        Server-->>Client: 200 OK
    else rate limited
        Server-->>Client: 429 Too Many Requests
    end
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
sequenceDiagram
    participant Module as Other Modules
    participant Client as aw-notify-client
    participant Server as aw-notify /notify
    participant NotifySys as OS Notification

    Module->>Client: send notification request
    Client->>Server: POST /notify (hardened endpoint)
    Server->>Server: check rate limit / min delay
    alt within rate limit
        Server->>NotifySys: display notification
        Server-->>Client: 200 OK
    else rate limited
        Server-->>Client: 429 Too Many Requests
    end
Loading

Reviews (1): Last reviewed commit: "build(deps): bump aw-notify from `f63810..." | Re-trigger Greptile

@ErikBjare

Copy link
Copy Markdown
Member

Not merging due to ActivityWatch/aw-notify-rs@352e01b which adds a HTTP server. Tracked in ActivityWatch/aw-notify-rs#27

@ErikBjare ErikBjare left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file submodules Pull requests that update Submodules code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant