Skip to content

feat: GitHub integration#8

Open
SeanJA239 wants to merge 12 commits intomainfrom
github-integration
Open

feat: GitHub integration#8
SeanJA239 wants to merge 12 commits intomainfrom
github-integration

Conversation

@SeanJA239
Copy link
Copy Markdown
Collaborator

No description provided.

AprilNEA and others added 11 commits February 26, 2026 17:18
feat: debounce every 500ms
fix debounce and env var config
show update details & comments in cards
Rename package to larkstack, bump edition from 2021 to 2024,
add authors/license fields, and split LICENSE into MIT + Apache-2.0.
Split single serial job into parallel fmt/clippy/test jobs,
add concurrency control to cancel stale runs, and skip CI
on non-Rust file changes.
Introduce a three-layer architecture (sources/ → Event → sinks/) so
that adding a new source or sink requires no changes to the other layer.

- Add `event.rs` with `Priority` enum and `Event` enum as the middle layer
- Add `dispatch.rs` to route events to all registered sinks
- Move Linear webhook models, client, and handler into `sources/linear/`
- Move Lark card builders, bot client, and webhook into `sinks/lark/`
- Generalize `debounce.rs` to hold `Event` instead of `Issue`
- Rewrite all comments to follow rustdoc conventions (//! and ///)
- Add `.claude/skills/rust-doc-comments` for reusable doc audit
Replace scattered env::var() calls with declarative LinearConfig,
LarkConfig, and ServerConfig structs loaded via figment's Env provider.
- Strip emoji headings and promotional language
- Cut significance inflation and copula avoidance
- Simplify boldface inline-header lists
- Tone down copy to be factual and direct
Introduce a `cf-worker` feature flag that compiles the same codebase as a
Cloudflare Worker (wasm32-unknown-unknown cdylib). Debounce semantics are
preserved via a Durable Object with alarms replacing the in-memory
DebounceMap + tokio::spawn pattern.

Key changes:
- Restructure into lib + bin dual target
- Cfg-gate config, debounce, handler, and bot modules
- Decouple sinks from AppState for reuse in DO alarm handler
- Add DebounceObject Durable Object implementation
- Manual request dispatch in CF entry to bypass axum Handler Send bounds
- Guard against simultaneous native + cf-worker feature activation
- Add deployment documentation for Cloudflare Workers
Move environment variable reference to docs/configuration.md and Railway
deployment steps to docs/deploy-railway.md. READMEs now serve as a concise
overview with links to the detailed guides. Updated badges and descriptions
to reflect Cloudflare Workers support.
Copy link
Copy Markdown
Contributor

@AprilNEA AprilNEA left a comment

Choose a reason for hiding this comment

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

use octocrab instead of building yourself

Comment thread src/sinks/lark/bot.rs Outdated
"receive_id": chat_id,
"msg_type": "interactive",
"content": serde_json::to_string(card).unwrap_or_default(),
});
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

use struct

@AprilNEA
Copy link
Copy Markdown
Contributor

AprilNEA commented Mar 2, 2026

@SeanJA239 SeanJA239 force-pushed the github-integration branch from 6e49cbc to 1526f5d Compare March 9, 2026 10:35
- Introduced octocrab models for type-safe GitHub webhook handling
- Replaced json!() macros with proper struct serialization

refactor: introduced octocrab models for Github webhook
@SeanJA239 SeanJA239 force-pushed the github-integration branch from 1526f5d to 4b07699 Compare March 13, 2026 15:34
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.

2 participants