Skip to content

chore: mark t005 multi-tenant credential storage as complete #227

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
marcusquinn merged 2 commits into from
Jan 25, 2026
Merged

chore: mark t005 multi-tenant credential storage as complete #227

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
53c0d25
plan: add t079 - consolidate Plan+ and AI-DevOps into Build+
marcusquinn Jan 25, 2026
8ebd3e1
chore: mark t005 multi-tenant credential storage as complete
marcusquinn Jan 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 20 additions & 2 deletions TODO.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ Tasks with no open blockers - ready to work on. Use `/ready` to refresh this lis
- [ ] t009 Claude Code Destructive Command Hooks #plan → [todo/PLANS.md#claude-code-destructive-command-hooks] ~1h (ai:30m test:20m read:10m) logged:2025-12-21
- [ ] t008 aidevops-opencode Plugin #plan → [todo/PLANS.md#aidevops-opencode-plugin] ~4h (ai:2h test:1h read:1h) logged:2025-12-21
- [x] t004 Add Ahrefs MCP server integration #seo ~4h (ai:2h test:1h read:1h) logged:2025-12-20 completed:2026-01-25
- [ ] t005 Implement multi-tenant credential storage #security ~1.5d (ai:8h test:4h read:2h) logged:2025-12-20
- [x] t005 Implement multi-tenant credential storage #security ~1.5d (ai:8h test:4h read:2h) logged:2025-12-20 completed:2026-01-24
Copy link
Contributor

@coderabbitai coderabbitai bot Jan 25, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Move completed task t005 from Backlog to Done section.

Task t005 is marked complete but remains in the Backlog section. Per the TODO.md format, completed tasks should be relocated to the Done section (starting at line 294).

📋 Recommended fix
  1. Remove line 67 from the Backlog section
  2. Add to the Done section (after line 296):
+- [x] t005 Implement multi-tenant credential storage `#security` ~1.5d (ai:8h test:4h read:2h) logged:2025-12-20 completed:2026-01-24
+  - Notes: Implementation existed in v2.80.0 via PR `#178`. Multi-tenant credential storage allows secure isolation of credentials per tenant/workspace.
🤖 Prompt for AI Agents 
In `@TODO.md` at line 67, Move the completed task entry for "t005 Implement
multi-tenant credential storage" out of the Backlog and into the Done section:
delete the Backlog list item matching "t005 Implement multi-tenant credential
storage `#security` ~1.5d (ai:8h test:4h read:2h) logged:2025-12-20
completed:2026-01-24" and append the same line under the Done section (after the
Done header starting at line ~294) so the task appears in Done instead of
Backlog.

- [ ] t070 Backlink & Expired Domain Checker subagent #seo #domains ~45m (ai:30m test:10m read:5m) logged:2026-01-24
- Notes: Create subagent that: 1) Checks for lost/broken backlinks using Ahrefs API or DataForSEO backlinks endpoint. 2) Cross-references referring domains against expired domain checkers (evaluate: expired-domains.co API, expireddomains.net, or GitHub tools like @peterprototypes/expired-domains, @Jeongseup/expired-domain-finder). 3) Reports which referring domains have expired and may be available to purchase for link reclamation. Workflow: fetch backlink profile → identify lost/broken links → check domain expiry status → rank by DA/DR/traffic value → output purchase candidates. Add to seo/ or tools/seo/.
- [ ] t071 Voice AI models for speech generation and transcription #tools #voice #ai ~4h (ai:2h test:1h read:1h) logged:2026-01-24 related:t027
Expand Down Expand Up @@ -170,6 +170,24 @@ Tasks with no open blockers - ready to work on. Use `/ready` to refresh this lis
- Notes: Inspired by ALwrity persona system. Extend content/guidelines.md with platform-specific sections for LinkedIn, Instagram, YouTube. Define voice, tone, structure, and best practices per platform.
- [ ] t077 LinkedIn Content Subagent #tools #social-media ~1h (ai:45m test:15m) logged:2026-01-25 ref:t037
- Notes: Create tools/social-media/linkedin.md. Support post types: text posts, articles, carousels, documents. Follow bird.md pattern. Include LinkedIn-specific best practices (hashtags, timing, engagement).
- [ ] t079 Consolidate Plan+ and AI-DevOps into Build+ #refactor #agents #architecture ~4h (ai:3h test:1h) logged:2026-01-25
- Notes: Remove Plan+ and AI-DevOps as separate primary agents. Build+ becomes the single coding/devops agent.
- [ ] t079.1 Audit AI-DevOps for unique knowledge to merge into Build+ ~30m
- Notes: AI-DevOps has: framework operations (setup, troubleshooting, architecture), MCP integrations, OpenCode plugins, credential management. Most already accessible via subagents. Merge any unique content into Build+ or ensure subagent coverage.
- [ ] t079.2 Add intent detection to Build+ (deliberation vs execution) ~1h
- Notes: Build+ should detect message intent: "What do you think..." / "How should we..." → research/discuss mode (don't code yet). "Implement X" / "Fix Y" / "Add Z" → execution mode. Ambiguous → ask "implement now or discuss first?"
- [ ] t079.3 Merge Plan+ planning workflow into Build+ ~1h
- Notes: Plan+ has: parallel explore agents, investigation phases, synthesis, handoff protocol. Build+ should incorporate these as modes, not separate agent. Keep pre-edit git check as the safety gate (not agent separation).
- [ ] t079.4 Remove Plan+ from primary agents ~30m
- Notes: Remove from opencode.json agent list, delete plan-plus.md, update generate-opencode-agents.sh, remove Plan+ specific permissions/tools config.
- [ ] t079.5 Remove AI-DevOps from primary agents ~30m
- Notes: Keep aidevops.md as subagent (@aidevops) for framework-specific tasks. Remove from primary agent list. Users invoke via @aidevops when needed.
- [ ] t079.6 Update AGENTS.md and documentation ~30m
- Notes: Update greeting workflow (no Plan+ fallback needed), update agent switching docs, update onboarding to reflect single-agent model.
- [ ] t079.7 Test Build+ handles planning and execution modes ~30m
- Notes: Test: "What's the best approach for X?" (should research, not code). "Implement X" (should code). "Review this" (should analyze). Verify pre-edit check still prevents accidental main branch edits.
- [ ] t079.8 Update setup.sh and aidevops update to cleanup removed agents ~30m
- Notes: setup.sh must: 1) Remove Plan+ from opencode.json agent list, 2) Remove AI-DevOps from primary agents (keep as subagent), 3) Delete orphaned agent files from ~/.config/opencode/agent/, 4) Update default_agent to Build+. This ensures existing installs get cleaned up on `aidevops update`.
Comment on lines +173 to +190
Copy link

@gemini-code-assist gemini-code-assist bot Jan 25, 2026

Choose a reason for hiding this comment

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

medium

The addition of this new task t079 and its subtasks seems out of scope for this pull request, which is intended only to mark task t005 as complete. Including a large, unrelated change like this can create confusion and makes the commit history harder to follow. It would be best to move this new refactoring plan to a separate pull request.

- [x] t063 Fix secretlint scanning performance #bugfix #secretlint #performance ~30m (ai:15m test:10m read:5m) logged:2026-01-14 completed:2026-01-14
- Notes: Added python-env, .osgrep, .scannerwork to .secretlintignore. Added bun.lock to .gitignore to maintain subset rule. Increased Docker timeout 30s→60s. Optional: glob whitelist in linters-local.sh for further optimization.
- [x] t066 Add /add-skill command for external skill import #tools #skills #agents ~4h (ai:3h test:30m read:30m) logged:2026-01-21 started:2026-01-21T00:00Z completed:2026-01-21 actual:4h
Expand All @@ -194,7 +212,7 @@ t010,Evaluate Merging build-agent and build-mcp into aidevops,,plan|architecture
t009,Claude Code Destructive Command Hooks,,plan|claude|git|security,4h,2h,1h,1h,2025-12-21T12:00Z,pending,,,
t008,aidevops-opencode Plugin,,plan,2d,1d,0.5d,0.5d,2025-12-21T01:50Z,pending,,,
t004,Add Ahrefs MCP server integration,,seo,2d,1d,0.5d,0.5d,2025-12-20T00:00Z,pending,,,
t005,Implement multi-tenant credential storage,,security,5d,3d,1.5d,0.5d,2025-12-20T00:00Z,pending,,,
t005,Implement multi-tenant credential storage,,security,5d,3d,1.5d,0.5d,2025-12-20T00:00Z,done,2026-01-24,,
Copy link

@augmentcode augmentcode bot Jan 25, 2026

Choose a reason for hiding this comment

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

In the <!--TOON:backlog...{...,status,blocked_by,...}--> schema, the field immediately after status is labeled blocked_by, but this row now stores 2026-01-24 there. If any tooling relies on these column semantics, it may interpret that date as a blocker rather than completion metadata.

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎

Copy link
Contributor

@coderabbitai coderabbitai bot Jan 25, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Migrate t005 from backlog CSV to done CSV with correct schema.

Line 215 has multiple issues:

  1. Contains 13 fields but backlog schema expects 12 (extra 0.5d field)
  2. 2026-01-24 appears in the blocked_by field position (field 10 per schema at line 198)
  3. Task is marked done but remains in backlog CSV instead of done CSV

The done section has a different schema (line 330): {id,desc,owner,tags,est,actual,logged,started,completed,status}

🔧 Recommended fix
  1. Remove line 215 from the backlog CSV block
  2. Add to the done CSV block (after line 330 header, before the closing -->):
+t005,Implement multi-tenant credential storage,,security,1.5d,,2025-12-20T00:00Z,,2026-01-24,done

Note: If actual time or started date are known, include them in the appropriate positions.

🤖 Prompt for AI Agents 
In `@TODO.md` at line 215, Remove the erroneous backlog entry for task id "t005"
(the CSV row starting with "t005,Implement multi-tenant credential storage") and
migrate it into the done CSV section using the done-schema fields
{id,desc,owner,tags,est,actual,logged,started,completed,status}; ensure you drop
the extra `0.5d` field, move the `2026-01-24` value into the correct `completed`
(or `started`/`blocked_by` as appropriate) column per the done schema, and place
the new row after the done CSV header (line ~330) so the task is removed from
backlog and recorded correctly as done.

t006,Add Playwright MCP auto-setup to setup.sh,,browser,1d,0.5d,0.5d,,2025-12-20T00:00Z,done,,,
t007,Create MCP server for QuickFile accounting API,,accounting,3d,2d,1d,,2025-12-20T00:00Z,pending,,,
t012,OCR Invoice/Receipt Extraction Pipeline,,plan|accounting|ocr|automation,3d,1.5d,1d,0.5d,2025-12-21T22:00Z,pending,,,
Expand Down
Loading