Skip to content

docs: clarify Copilot CLI review command path usage#187

Merged
mahata merged 2 commits intomainfrom
chore/replace-security-auditor-with-copilot-cli
Apr 8, 2026
Merged

docs: clarify Copilot CLI review command path usage#187
mahata merged 2 commits intomainfrom
chore/replace-security-auditor-with-copilot-cli

Conversation

@mahata
Copy link
Copy Markdown
Owner

@mahata mahata commented Apr 8, 2026 •
edited by Copilot AI
Loading

The Copilot CLI review instructions in AGENTS.md used an ambiguous placeholder (@<filename>) and conflicted with wording around task-level vs file-level execution. This updates the guidance to be copy-paste safe and semantically consistent.

  • Documentation clarity

    • Replaced @<filename> with @path/to/file in the command template.
    • Added explicit instruction to replace the placeholder with a real repository path.
    • Added a concrete example path (hono/routes/health.ts) to reduce misuse.

  • Instruction alignment

    • Kept the “run after completing a task” framing.
    • Clarified that the command is executed for each created/modified file within that task, removing ambiguity.
copilot -p 'Review @path/to/file for security, efficiency, and readability. Suggest specific improvements.' --model gpt-5.4 -s
# e.g. @hono/routes/health.ts

@mahata
chore: replace security-auditor agent with Copilot CLI reviews
4014f9c 
Remove the dedicated security-auditor OpenCode agent and add a Copilot CLI
review step to AGENTS.md instead. This gives multi-model review coverage
(Opus 4.6 writes, GPT-5.4 reviews) and runs per-task rather than per-file
to keep the workflow efficient.
Copilot AI review requested due to automatic review settings April 8, 2026 03:21
Copilot AI reviewed Apr 8, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Replaces the OpenCode security-auditor subagent workflow with a documented “Copilot CLI Reviews” step, keeping review coverage while simplifying the .opencode/agents set.

Changes:

  • Added a “Copilot CLI Reviews” section to AGENTS.md describing how to run Copilot CLI reviews after completing a task.
  • Removed the .opencode/agents/security-auditor.md subagent definition.
Show a summary per file
File Description
AGENTS.md Documents the new Copilot CLI review workflow to replace the removed security-auditor agent.
.opencode/agents/security-auditor.md Deletes the security-auditor subagent configuration and instructions.

Copilot's findings

  • Files reviewed: 2/2 changed files
  • Comments generated: 2

Comment thread AGENTS.md
Comment on lines +159 to +161
```shell
copilot -p 'Review @<filename> for security, efficiency, and readability. Suggest specific improvements.' --model gpt-5.4 -s
```
Copy link

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

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

The placeholder @<filename> is likely to be copy/pasted literally. Consider switching to a placeholder that matches expected CLI usage (e.g., @path/to/file) and explicitly stating that the angle brackets should be replaced with the real path (or provide a concrete example command).

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Owner Author

@mahata mahata Apr 8, 2026

Choose a reason for hiding this comment

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

@copilot apply changes based on this feedback

Copy link
Copy Markdown
Contributor

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

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

Updated in 5af3428: the command now uses @path/to/file and the section explicitly tells readers to replace it with a real repo path, including a concrete example (hono/routes/health.ts).

Comment thread AGENTS.md Outdated
Comment on lines +157 to +163
After completing a task (not after every individual file edit), you must run:

```shell
copilot -p 'Review @<filename> for security, efficiency, and readability. Suggest specific improvements.' --model gpt-5.4 -s
```

for each file that was created or modified during the task. If Copilot finds issues, refactor the code and repeat the review for the affected files.
Copy link

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

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

The new section says reviews are run after completing a task, but then mandates running the command once per modified file. The PR description mentions “per-task (not per-file)”; please align the documentation (either describe a single per-task review command over all files, or remove the “not per-file” wording).

Copilot uses AI. Check for mistakes.
Copilot AI changed the title chore: replace security-auditor agent with Copilot CLI reviews docs: clarify Copilot CLI review command path usage Apr 8, 2026
@mahata mahata merged commit 9e783ad into main Apr 8, 2026
3 checks passed
@mahata mahata deleted the chore/replace-security-auditor-with-copilot-cli branch April 8, 2026 04:20
mahata added a commit that referenced this pull request Apr 8, 2026
@mahata
@dependabot
Add deployment tagging, GitHub Releases, PR template, and workspaces (#…
43986d9 
…194)

* feat: add deployment tagging, GitHub Releases, and PR template

Add a post-deploy step to deploy.yml that creates a git tag
(deploy-YYYY-MM-DD-SHA) and a GitHub Release with auto-generated
notes after each successful production deployment. This provides
a clear history of which commits were deployed and when.

Add a default PR template with a pre-deploy checklist covering
CI status, migration review, secrets, and Durable Objects state.

* Update .github/workflows/deploy.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: scope contents:write permission to deploy job only

Agent-Logs-Url: https://github.com/mahata/mlack/sessions/b28d6893-dc36-48da-a24d-589cb4ad8e05

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* deps-dev(deps-dev): bump @cloudflare/workers-types (#164)

Bumps [@cloudflare/workers-types](https://github.com/cloudflare/workerd) from 4.20260317.1 to 4.20260329.1.
- [Release notes](https://github.com/cloudflare/workerd/releases)
- [Changelog](https://github.com/cloudflare/workerd/blob/main/RELEASE.md)
- [Commits](https://github.com/cloudflare/workerd/commits)

---
updated-dependencies:
- dependency-name: "@cloudflare/workers-types"
  dependency-version: 4.20260329.1
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore: add conventional-commit skill for Copilot

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* deps(deps): bump hono in the production-dependencies group (#168)

Bumps the production-dependencies group with 1 update: [hono](https://github.com/honojs/hono).


Updates `hono` from 4.12.9 to 4.12.11
- [Release notes](https://github.com/honojs/hono/releases)
- [Commits](honojs/hono@v4.12.9...v4.12.11)

---
updated-dependencies:
- dependency-name: hono
  dependency-version: 4.12.11
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: production-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* deps(deps): bump the development-dependencies group with 11 updates (#169)

Bumps the development-dependencies group with 11 updates:

| Package | From | To |
| --- | --- | --- |
| [@biomejs/biome](https://github.com/biomejs/biome/tree/HEAD/packages/@biomejs/biome) | `2.4.9` | `2.4.10` |
| [@playwright/test](https://github.com/microsoft/playwright) | `1.58.2` | `1.59.1` |
| [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) | `24.12.0` | `24.12.2` |
| [@biomejs/cli-darwin-arm64](https://github.com/biomejs/biome/tree/HEAD/packages/@biomejs/biome) | `2.4.9` | `2.4.10` |
| [@biomejs/cli-darwin-x64](https://github.com/biomejs/biome/tree/HEAD/packages/@biomejs/biome) | `2.4.9` | `2.4.10` |
| [@biomejs/cli-linux-arm64-musl](https://github.com/biomejs/biome/tree/HEAD/packages/@biomejs/biome) | `2.4.9` | `2.4.10` |
| [@biomejs/cli-linux-arm64](https://github.com/biomejs/biome/tree/HEAD/packages/@biomejs/biome) | `2.4.9` | `2.4.10` |
| [@biomejs/cli-linux-x64-musl](https://github.com/biomejs/biome/tree/HEAD/packages/@biomejs/biome) | `2.4.9` | `2.4.10` |
| [@biomejs/cli-linux-x64](https://github.com/biomejs/biome/tree/HEAD/packages/@biomejs/biome) | `2.4.9` | `2.4.10` |
| [@biomejs/cli-win32-arm64](https://github.com/biomejs/biome/tree/HEAD/packages/@biomejs/biome) | `2.4.9` | `2.4.10` |
| [@biomejs/cli-win32-x64](https://github.com/biomejs/biome/tree/HEAD/packages/@biomejs/biome) | `2.4.9` | `2.4.10` |


Updates `@biomejs/biome` from 2.4.9 to 2.4.10
- [Release notes](https://github.com/biomejs/biome/releases)
- [Changelog](https://github.com/biomejs/biome/blob/main/packages/@biomejs/biome/CHANGELOG.md)
- [Commits](https://github.com/biomejs/biome/commits/@biomejs/biome@2.4.10/packages/@biomejs/biome)

Updates `@playwright/test` from 1.58.2 to 1.59.1
- [Release notes](https://github.com/microsoft/playwright/releases)
- [Commits](microsoft/playwright@v1.58.2...v1.59.1)

Updates `@types/node` from 24.12.0 to 24.12.2
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

Updates `@biomejs/cli-darwin-arm64` from 2.4.9 to 2.4.10
- [Release notes](https://github.com/biomejs/biome/releases)
- [Changelog](https://github.com/biomejs/biome/blob/main/packages/@biomejs/biome/CHANGELOG.md)
- [Commits](https://github.com/biomejs/biome/commits/@biomejs/biome@2.4.10/packages/@biomejs/biome)

Updates `@biomejs/cli-darwin-x64` from 2.4.9 to 2.4.10
- [Release notes](https://github.com/biomejs/biome/releases)
- [Changelog](https://github.com/biomejs/biome/blob/main/packages/@biomejs/biome/CHANGELOG.md)
- [Commits](https://github.com/biomejs/biome/commits/@biomejs/biome@2.4.10/packages/@biomejs/biome)

Updates `@biomejs/cli-linux-arm64-musl` from 2.4.9 to 2.4.10
- [Release notes](https://github.com/biomejs/biome/releases)
- [Changelog](https://github.com/biomejs/biome/blob/main/packages/@biomejs/biome/CHANGELOG.md)
- [Commits](https://github.com/biomejs/biome/commits/@biomejs/biome@2.4.10/packages/@biomejs/biome)

Updates `@biomejs/cli-linux-arm64` from 2.4.9 to 2.4.10
- [Release notes](https://github.com/biomejs/biome/releases)
- [Changelog](https://github.com/biomejs/biome/blob/main/packages/@biomejs/biome/CHANGELOG.md)
- [Commits](https://github.com/biomejs/biome/commits/@biomejs/biome@2.4.10/packages/@biomejs/biome)

Updates `@biomejs/cli-linux-x64-musl` from 2.4.9 to 2.4.10
- [Release notes](https://github.com/biomejs/biome/releases)
- [Changelog](https://github.com/biomejs/biome/blob/main/packages/@biomejs/biome/CHANGELOG.md)
- [Commits](https://github.com/biomejs/biome/commits/@biomejs/biome@2.4.10/packages/@biomejs/biome)

Updates `@biomejs/cli-linux-x64` from 2.4.9 to 2.4.10
- [Release notes](https://github.com/biomejs/biome/releases)
- [Changelog](https://github.com/biomejs/biome/blob/main/packages/@biomejs/biome/CHANGELOG.md)
- [Commits](https://github.com/biomejs/biome/commits/@biomejs/biome@2.4.10/packages/@biomejs/biome)

Updates `@biomejs/cli-win32-arm64` from 2.4.9 to 2.4.10
- [Release notes](https://github.com/biomejs/biome/releases)
- [Changelog](https://github.com/biomejs/biome/blob/main/packages/@biomejs/biome/CHANGELOG.md)
- [Commits](https://github.com/biomejs/biome/commits/@biomejs/biome@2.4.10/packages/@biomejs/biome)

Updates `@biomejs/cli-win32-x64` from 2.4.9 to 2.4.10
- [Release notes](https://github.com/biomejs/biome/releases)
- [Changelog](https://github.com/biomejs/biome/blob/main/packages/@biomejs/biome/CHANGELOG.md)
- [Commits](https://github.com/biomejs/biome/commits/@biomejs/biome@2.4.10/packages/@biomejs/biome)

---
updated-dependencies:
- dependency-name: "@biomejs/biome"
  dependency-version: 2.4.10
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: development-dependencies
- dependency-name: "@playwright/test"
  dependency-version: 1.59.1
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: development-dependencies
- dependency-name: "@types/node"
  dependency-version: 24.12.2
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: development-dependencies
- dependency-name: "@biomejs/cli-darwin-arm64"
  dependency-version: 2.4.10
  dependency-type: indirect
  update-type: version-update:semver-patch
  dependency-group: development-dependencies
- dependency-name: "@biomejs/cli-darwin-x64"
  dependency-version: 2.4.10
  dependency-type: indirect
  update-type: version-update:semver-patch
  dependency-group: development-dependencies
- dependency-name: "@biomejs/cli-linux-arm64-musl"
  dependency-version: 2.4.10
  dependency-type: indirect
  update-type: version-update:semver-patch
  dependency-group: development-dependencies
- dependency-name: "@biomejs/cli-linux-arm64"
  dependency-version: 2.4.10
  dependency-type: indirect
  update-type: version-update:semver-patch
  dependency-group: development-dependencies
- dependency-name: "@biomejs/cli-linux-x64-musl"
  dependency-version: 2.4.10
  dependency-type: indirect
  update-type: version-update:semver-patch
  dependency-group: development-dependencies
- dependency-name: "@biomejs/cli-linux-x64"
  dependency-version: 2.4.10
  dependency-type: indirect
  update-type: version-update:semver-patch
  dependency-group: development-dependencies
- dependency-name: "@biomejs/cli-win32-arm64"
  dependency-version: 2.4.10
  dependency-type: indirect
  update-type: version-update:semver-patch
  dependency-group: development-dependencies
- dependency-name: "@biomejs/cli-win32-x64"
  dependency-version: 2.4.10
  dependency-type: indirect
  update-type: version-update:semver-patch
  dependency-group: development-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* deps(deps): bump tinyglobby from 0.2.14 to 0.2.15 (#170)

Bumps [tinyglobby](https://github.com/SuperchupuDev/tinyglobby) from 0.2.14 to 0.2.15.
- [Release notes](https://github.com/SuperchupuDev/tinyglobby/releases)
- [Changelog](https://github.com/SuperchupuDev/tinyglobby/blob/main/CHANGELOG.md)
- [Commits](SuperchupuDev/tinyglobby@0.2.14...0.2.15)

---
updated-dependencies:
- dependency-name: tinyglobby
  dependency-version: 0.2.15
  dependency-type: indirect
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* deps(deps): bump @emnapi/runtime from 1.9.1 to 1.9.2 (#172)

Bumps [@emnapi/runtime](https://github.com/toyobayashi/emnapi) from 1.9.1 to 1.9.2.
- [Release notes](https://github.com/toyobayashi/emnapi/releases)
- [Commits](toyobayashi/emnapi@v1.9.1...v1.9.2)

---
updated-dependencies:
- dependency-name: "@emnapi/runtime"
  dependency-version: 1.9.2
  dependency-type: indirect
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* deps(deps): bump vite from 7.0.5 to 7.3.1 (#171)

Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 7.0.5 to 7.3.1.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v7.3.1/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-version: 7.3.1
  dependency-type: indirect
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* deps(deps): bump tinyspy from 4.0.3 to 4.0.4 (#173)

Bumps [tinyspy](https://github.com/tinylibs/tinyspy) from 4.0.3 to 4.0.4.
- [Release notes](https://github.com/tinylibs/tinyspy/releases)
- [Commits](tinylibs/tinyspy@v4.0.3...v4.0.4)

---
updated-dependencies:
- dependency-name: tinyspy
  dependency-version: 4.0.4
  dependency-type: indirect
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* deps-dev(deps-dev): bump @cloudflare/workers-types (#174)

Bumps [@cloudflare/workers-types](https://github.com/cloudflare/workerd) from 4.20260329.1 to 4.20260405.1.
- [Release notes](https://github.com/cloudflare/workerd/releases)
- [Changelog](https://github.com/cloudflare/workerd/blob/main/RELEASE.md)
- [Commits](https://github.com/cloudflare/workerd/commits)

---
updated-dependencies:
- dependency-name: "@cloudflare/workers-types"
  dependency-version: 4.20260405.1
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* deps-dev(deps-dev): bump wrangler from 4.77.0 to 4.79.0 (#178)

Bumps [wrangler](https://github.com/cloudflare/workers-sdk/tree/HEAD/packages/wrangler) from 4.77.0 to 4.79.0.
- [Release notes](https://github.com/cloudflare/workers-sdk/releases)
- [Commits](https://github.com/cloudflare/workers-sdk/commits/wrangler@4.79.0/packages/wrangler)

---
updated-dependencies:
- dependency-name: wrangler
  dependency-version: 4.79.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* deps(deps): bump drizzle-orm from 0.45.1 to 0.45.2 (#165)

Bumps [drizzle-orm](https://github.com/drizzle-team/drizzle-orm) from 0.45.1 to 0.45.2.
- [Release notes](https://github.com/drizzle-team/drizzle-orm/releases)
- [Commits](drizzle-team/drizzle-orm@0.45.1...0.45.2)

---
updated-dependencies:
- dependency-name: drizzle-orm
  dependency-version: 0.45.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix: resolve duplicate @types/node@24.12.2 in pnpm-lock.yaml (#175)

* deps(deps): bump expect-type from 1.2.2 to 1.3.0

Bumps [expect-type](https://github.com/mmkal/expect-type) from 1.2.2 to 1.3.0.
- [Release notes](https://github.com/mmkal/expect-type/releases)
- [Commits](mmkal/expect-type@v1.2.2...v1.3.0)

---
updated-dependencies:
- dependency-name: expect-type
  dependency-version: 1.3.0
  dependency-type: indirect
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix: resolve duplicate @types/node@24.12.2 in pnpm-lock.yaml by updating @types/node to ^24.12.2

Agent-Logs-Url: https://github.com/mahata/mlack/sessions/75387082-98f8-4f32-a101-e17460603972

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* chore: merge main into branch, resolve @playwright/test version conflict

Agent-Logs-Url: https://github.com/mahata/mlack/sessions/93366234-814a-46ee-b2fd-3d5296eeaa11

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* feat: add mlack.uk custom domain route to production

* fix: resolve duplicate @types/node@24.12.2 in pnpm-lock.yaml (#176)

* deps(deps): bump postcss from 8.5.6 to 8.5.8

Bumps [postcss](https://github.com/postcss/postcss) from 8.5.6 to 8.5.8.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss@8.5.6...8.5.8)

---
updated-dependencies:
- dependency-name: postcss
  dependency-version: 8.5.8
  dependency-type: indirect
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix: resolve duplicate @types/node@24.12.2 in pnpm-lock.yaml after merging main

Agent-Logs-Url: https://github.com/mahata/mlack/sessions/ee849a11-8ea0-46b7-8001-8e30404fc116

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* deps(deps): bump vite from 7.0.5 to 7.3.2 (#183)

Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 7.0.5 to 7.3.2.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/v7.3.2/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v7.3.2/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-version: 7.3.2
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat: add Electron desktop client (#184)

* feat: add Electron desktop client with notifications and system tray

Add an Electron wrapper that loads the existing mlack web app in a
native desktop window. Includes:
- System tray icon with show/quit context menu
- Native OS notifications for new messages when window is unfocused
- WebSocket message interception via injected script
- Persistent cookie sessions across app restarts
- electron-builder config for macOS, Windows, and Linux packaging
- Workspace integration with root-level electron:dev/build/package scripts

* Update electron/src/notifications.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update electron/src/tray.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update electron/src/preload.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: disable electron-builder publish to prevent channel crash (#186)

* deps-dev(deps-dev): bump electron from 35.7.5 to 39.8.5 (#185)

Bumps [electron](https://github.com/electron/electron) from 35.7.5 to 39.8.5.
- [Release notes](https://github.com/electron/electron/releases)
- [Commits](electron/electron@v35.7.5...v39.8.5)

---
updated-dependencies:
- dependency-name: electron
  dependency-version: 39.8.5
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* docs: clarify Copilot CLI review command path usage (#187)

* chore: replace security-auditor agent with Copilot CLI reviews

Remove the dedicated security-auditor OpenCode agent and add a Copilot CLI
review step to AGENTS.md instead. This gives multi-model review coverage
(Opus 4.6 writes, GPT-5.4 reviews) and runs per-task rather than per-file
to keep the workflow efficient.

* docs: clarify copilot CLI file path placeholder

Agent-Logs-Url: https://github.com/mahata/mlack/sessions/8cef0ef9-23e7-4daf-9bc5-6af13accf40c

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* deps(deps): bump hono from 4.12.11 to 4.12.12 (#189)

Bumps [hono](https://github.com/honojs/hono) from 4.12.11 to 4.12.12.
- [Release notes](https://github.com/honojs/hono/releases)
- [Commits](honojs/hono@v4.12.11...v4.12.12)

---
updated-dependencies:
- dependency-name: hono
  dependency-version: 4.12.12
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix: enforce existing Git tag guard in electron release workflow (#188)

* feat: add GitHub Actions workflow for Electron desktop releases

Add a manual-dispatch workflow that builds MLack Desktop binaries for
macOS (.dmg), Windows (.nsis), and Linux (.AppImage) in parallel, then
creates a GitHub Release with all artifacts attached.

Includes version validation, duplicate release check, branch guard
(main only), scoped permissions, and concurrency protection.

* fix: validate existing git tags in electron release workflow

Agent-Logs-Url: https://github.com/mahata/mlack/sessions/9f1592bd-764d-4005-8c02-81a5c88659a2

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* fix: check existing release tags with git ls-remote

Agent-Logs-Url: https://github.com/mahata/mlack/sessions/9f1592bd-764d-4005-8c02-81a5c88659a2

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* fix: resolve Electron release build failures (#190)

* fix: resolve Electron release build failures

- Replace 16x16 placeholder icon with 512x512 PNG and generate .icns
  (macOS) and .ico (Windows) files to meet electron-builder requirements
- Move version update step after dependency install so npm is available
  from the project's Node.js version rather than the runner default

* Update .github/workflows/electron-release.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: write explicit newline when updating Electron package version in release workflow (#191)

* fix: avoid lifecycle scripts when updating electron package version

Replace pnpm version with a direct JSON file write to prevent Electron
from launching during the version update step, which fails on Linux CI
due to SUID sandbox configuration.

* fix: use explicit newline char when writing electron package version

Agent-Logs-Url: https://github.com/mahata/mlack/sessions/b04db865-e846-4b04-8b58-959693015c66

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* docs: align README Node.js prerequisite with Node 24 upgrade (#192)

* chore: upgrade Node.js to 24 and remove corepack

Node.js 24 no longer bundles corepack, so remove the corepack enable
steps from all CI workflows and drop the packageManager field from
package.json. Update .node-version and AGENTS.md to reflect Node 24.

* fix: install pnpm via pnpm/action-setup in CI workflows

Node.js 24 no longer bundles corepack, so pnpm must be installed
explicitly. Add pnpm/action-setup@v4 before actions/setup-node in
every CI job.

* docs: update README Node.js prerequisite to v24

Agent-Logs-Url: https://github.com/mahata/mlack/sessions/21f73092-2de8-4e30-b7eb-4dd8f51484dc

Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>

* feat: add workspaces with isolated channels, members, and invite system (#193)

* feat: add workspaces with isolated channels, members, and invite system

Introduce workspace support with path-based routing (/w/:slug/...), per-workspace
admin roles, invite links with 1-hour expiration, and one Durable Object per
workspace for WebSocket connections. Existing data auto-migrates into a seeded
'default' workspace. Migration uses PRAGMA defer_foreign_keys to handle the
channels table rebuild safely on D1.

* fix: address PR review feedback — type safety, SQL efficiency, and invite security

- Make workspace/workspaceMember optional in Variables type to match runtime
- First user auto-joining default workspace now becomes admin (bootstrap fix)
- Remove unreachable auth check in /w/:slug (middleware already handles it)
- Replace select-all-then-filter with SQL inner join for workspace queries
- Use subquery in seed.sql instead of hardcoded workspace_id=1
- Convert invite acceptance from mutating GET to GET confirmation + POST

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mahata <23497+mahata@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

+1 more reviewer

Copilot code review Copilot Copilot left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mahata