Uh oh!
There was an error while loading. Please reload this page.
Adding "ignore folder" configuration - #48
Closed
davidturissini wants to merge 2 commits into
Closed
Conversation
adamdotdevin
commented
May 23, 2025
Member
thdxr
commented
Jun 14, 2025
Member
closing as we rewrote the codebase - appreciate it though! |
randomm referenced
this pull request
in randomm/opencode
Jan 9, 2026
…49) * feat(#46): add database query delegation protocol for research agent Enable research agent to get database insights via delegation to postgres specialist, maintaining security boundaries. Research agent additions: - Database investigation protocol section - Clear delegation format and examples - List of allowed query types - DO NOT attempt direct psql access Postgres agent additions: - Query delegation acceptance protocol - Query validation rules (read-only only) - Execution safety guidelines - Example delegation handling Research basis: Multi-agent systems best practice shows delegation patterns are superior to direct access for security and auditability. Single audit point maintained (only postgres has DB access). Files: 2 prompts * fix(#48): add database command denials to enforce delegation pattern Added explicit database command denials (psql*, mysql*, mongosh*, redis-cli*, sqlite3*) to bash permissions for 11 agents to ensure all database operations go through the proper delegation channels. This security fix ensures a single audit point for all database interactions and prevents agents from bypassing the established delegation protocols.
andreipromarketing-dev pushed a commit
to andreipromarketing-dev/opencode
that referenced
this pull request
Apr 7, 2026
…malyco#48) Adds comprehensive database-reviewer agent with Supabase PostgreSQL best practices
bussard76 pushed a commit
to bussard76/openwork
that referenced
this pull request
May 12, 2026
…es (anomalyco#48) * refactor: extract session store * refactor: move workspace and extensions state into stores
grenade added a commit
to grenade/opencode
that referenced
this pull request
Jun 17, 2026
…very helexa is a self-hosted, distributed mesh that is not in the public models.dev catalog and whose model lists, limits, and cost change frequently. Add a first-class built-in provider so users never hand-edit opencode.json model tables: - Seed a minimal helexa provider into the catalog so setting HELEXA_ENDPOINT (or a provider baseURL) alone activates it — no opencode.json provider stanza needed. Rides @ai-sdk/openai-compatible. - Add a custom() loader whose discoverModels() fetches helexa's live /v1/models at the resolved endpoint and maps per-model token limits and cost onto opencode models, so context %, compaction headroom, and spend work with zero per-model config. Mapping verified against a live cortex gateway's /v1/models. - Require an explicit endpoint (baseURL option or HELEXA_ENDPOINT); no hardcoded default until a canonical public router exists. - Generalize the model-discovery invocation, which was hardcoded to gitlab, to run for any provider that registers a discoverModels loader. Refs helexa#61. Builds on helexa#60 (error envelope), anomalyco#48 (usage), anomalyco#62 (catalog limit/cost). Co-Authored-By: Claude <noreply@anthropic.com>
6 tasks
zoulukuang added a commit
to zoulukuang/deskfox
that referenced
this pull request
Aug 14, 2026
…diff: 单个新文件,一份不可拆的执行脚本] 新增 `run_group4.py`,覆盖 anomalyco#40~anomalyco#50,全部通过。样本由 `make_fixtures.py` 生成、内容带特征词, 所以「预览对不对」是断言出来的,不是「看着像渲染出来了」。 ## 被实测推翻的「想当然」 - **代码/文本预览渲染在 `<diffs-container>` 的 shadow DOM 里**。 `document.body.innerText` 里根本没有文件内容 —— 于是「打开 plain.txt 后搜不到 TXTMARK」 一度被误读成「代码类文件没渲染」,还去翻了 iframe、canvas、闭合 shadow。 内容一直都在,只是选择器不穿 shadow 边界。已给 uiprobe 加 `deep_find_text` / `selection_text`。 - **对已打开的文件再点一次是「收起预览」**(界面上明写着「点击可收起预览」)。 脚本连开两次同一文件 = 把预览关掉,这是上面那个「搜不到内容」的直接触发条件。 `open_tree_path` 现在会核对预览是否真开着,被收起就再点回来。 - **右键菜单项是普通 `button`,不是 `role=menuitem`**;且**空白处右键给不出菜单** —— 「添加到聊天窗口 / 复制 / 导出为 Word」是针对选区的,必须先拖选。 第一版没选就右键、又按 ARIA role 查,拿到空菜单,差点判成缺陷。 - **`.xlsx` 走 LibreOffice 分页渲染,没有 `<table>`**。第一版按「有没有 table」写, 实际 `tables=0` 却因为文本命中而报了通过 —— 是**断言写松了蒙对**,已改成 断言两个 sheet 的特征词都在(证明整本工作簿都转换了)。 - **`code/` 目录初始就是展开的**,按「先变多再变回」写的断言直接误判(17→12→17); 改成只要求两次点击一开一合、且回到原状,不假设初始态。 ## anomalyco#48 的结论:能力齐全,是条目把入口写死了 右键菜单里**只有「关闭其他标签」**;「关闭标签页」在命令面板/⌘W; 「重新打开已关闭的标签页」命令面板搜不到、`mod+shift+t` 也不响应。 去基准版 `e77443750e`(合上游前)对照:**源码里根本没有 `reopenClosed`** —— 它是上游本次**新增**的命令,注册在 `titlebar.tsx` 的 tab 条上,而 DeskFox 的标签条 不走那条渲染路径。故属「上游新功能未接入」,**不是同步弄丢了功能**,按事实记录不报 FAIL。 CHECKLIST 同步更正 anomalyco#44 / anomalyco#47 / anomalyco#48 三条的预期与工具列。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Wut6R6htBzR8fBN8gbJkWg
zoulukuang added a commit
to zoulukuang/deskfox
that referenced
this pull request
Aug 14, 2026
第 4 组 11/11 全过;第 5~7 组 11 通过 2 跳过(飞书需真实账号)。 GetBot 首位 + 推荐标(fork 最核心定制点)已验。 单列本轮修掉的真缺陷:文件树点文件后焦点丢失 —— 根因是行成为 active 后被 Tooltip 分支重挂、DOM 节点销毁重建,把 handleClick 里设的焦点冲掉;目录行不触发, 故当初「只修好一半」。commit 1f458b9。 anomalyco#48 判为**非回归**:重开已关闭标签是上游本次新增命令,基准版 e774437 源码里 根本没有 reopenClosed,只是本布局未接入。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Wut6R6htBzR8fBN8gbJkWg
zoulukuang added a commit
to zoulukuang/deskfox
that referenced
this pull request
Aug 14, 2026
…-toolkit] 自动化原理上做不到的 10 条收成 `MANUAL-CHECKLIST.md`,每条给出前置/步骤/通过判据/ 失败记什么,并**必须写明「机器为什么做不到」** —— 否则下次会有人重复写一版跑不通的脚本。 单列第二节「本可自动化但本轮未覆盖」8 条(anomalyco#16 面板矩阵 / anomalyco#22 终端 / anomalyco#26 聊天引用 / anomalyco#27 md 内链拦截,以及只验了入口没验效果的 anomalyco#59anomalyco#62anomalyco#63anomalyco#55)。 它们**不是「验过了」而是「还没验」**,混进人工单里会被误当成没问题,故显式分开。 更正 anomalyco#12:总清单原文写「崩溃/无响应恢复对话框 —— 出现且按钮可用」, 但 renderer-crash-guard.ts 的实现是**静默自愈**(120s 内第二次可数崩溃 → 隔离 .dat 快照 → reload,单次崩溃只记日志),根本没有对话框。同 anomalyco#21 / anomalyco#48,属条目本身写错,就地更正并注明源码依据。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Wut6R6htBzR8fBN8gbJkWg
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Today,
opencodehas a hard coded list of "common" folders that it skips when greping a code base. This PR exposes "ignored" folders as a configuration so users can have control over which foldersopencodeskips.Configuration
{ "ignore": { "folders": ["path/to/ignored-folder"] } }There doesn't seem to be any tests for this, so I didn't add any but I did verify that the configuration is being loaded correctly.
If this is accepted, it would probably be desirable to move
commonIgnoredDirscloser to the configuration parsing.