Skip to content

docs: fix plugin init help text - add missing [namespace] argument (t308)#1191

Merged
marcusquinn merged 1 commit intomainfrom
feature/t308
Feb 12, 2026
Merged

docs: fix plugin init help text - add missing [namespace] argument (t308)#1191
marcusquinn merged 1 commit intomainfrom
feature/t308

Conversation

@marcusquinn
Copy link
Owner

@marcusquinn marcusquinn commented Feb 12, 2026

Summary

Fix help text for aidevops plugin init command to include the missing [namespace] argument.

Details

CodeRabbit finding from PR#763 identified that the help text was incomplete:

  • Updated command description from init [dir] [name] to init [dir] [name] [namespace]
  • Updated example from aidevops plugin init ./my-plugin my-plugin to aidevops plugin init ./my-plugin my-plugin my-plugin

The help text now matches the actual command signature (lines 2472-2474 in aidevops.sh).

Testing

  • bash -n aidevops.sh - syntax OK
  • Verified help text matches implementation signature

Notes

ShellCheck could not run due to library dependency issue (libgmp.10.dylib not found), but bash syntax check passed.

Fixes: t308
Ref #1181

Summary by CodeRabbit

Release Notes

  • New Features

    • The plugin init command now accepts an optional namespace parameter, providing enhanced flexibility during plugin initialization.
  • Documentation

    • Updated help text and usage examples for the plugin init command.

@gemini-code-assist
Copy link

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 12, 2026

Walkthrough

The plugin init subcommand in aidevops.sh is extended to accept an optional namespace parameter, changing the command signature from init <dir> <name> to init <dir> <name> <namespace>. Namespace validation logic previously performed during init is removed, with namespace computation retained for default assignment when not explicitly provided.

Changes

Cohort / File(s) Summary
Plugin Init Namespace Enhancement
aidevops.sh
Extended cmd_plugin init subcommand to accept optional [namespace] parameter; removed strict namespace validation block; updated help text and usage examples to reflect new signature.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related issues

  • t308: Fix help text in aidevops.sh #1181: Directly addresses the objective of adding an optional [namespace] parameter to the init subcommand and updating help text documentation to reflect this new parameter.

Possibly related PRs

Poem

Namespaces now dance to your tune, 🎭
Optional, flexible, arriving just in time,
Init commands simplified, no longer austere,
Validation laid to rest with grace,
Your scaffolding blooms with choice. ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating help text for the plugin init command to document the missing [namespace] argument.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/t308

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
aidevops.sh (1)

2592-2592: Consider a more illustrative example that demonstrates a distinct namespace.

Since namespace defaults to the plugin name (line 2474), passing my-plugin three times doesn't show why someone would use the third argument. A distinct namespace value would better demonstrate the feature:

💡 Suggested example
-		echo "  aidevops plugin init ./my-plugin my-plugin my-plugin"
+		echo "  aidevops plugin init ./my-plugin my-plugin custom-ns"

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

CodeRabbit finding from PR#763 - help text was missing the third positional
argument for the plugin init command. Updated both the command description
and the example to include [namespace].
@github-actions
Copy link

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

�[0;34m[INFO]�[0m Latest Quality Status:
SonarCloud: 0 bugs, 0 vulnerabilities, 15 code smells

�[0;34m[INFO]�[0m Recent monitoring activity:
Thu Feb 12 02:40:17 UTC 2026: Code review monitoring started
Thu Feb 12 02:40:17 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 15

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 15
  • VULNERABILITIES: 0

Generated on: Thu Feb 12 02:40:19 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

@sonarqubecloud
Copy link

marcusquinn added a commit that referenced this pull request Feb 12, 2026
Add escalating conflict resolution to rebase_sibling_pr():
1. Plain rebase (no conflicts)
2. Rebase with -Xtheirs (feature branch wins on conflicts)
3. AI CLI resolution for complex cases (Strategy 3 fallback)

Also adds:
- resolve_rebase_conflicts() for AI-assisted per-file resolution
- Stale worktree cleanup (abort stuck rebases, fix detached HEAD)
- Temp worktree creation when no worktree exists (avoids dirty tree)
- CONFLICTING added to t298 handler (was only BEHIND/DIRTY)
- Phase 7b: periodic retry of merge-conflict-blocked tasks (30min)

Tested: resolved 4 blocked PRs (#1171, #1187, #1188, #1191) that
were stuck with merge conflicts — all resolved with -Xtheirs.
@marcusquinn marcusquinn merged commit df72a1c into main Feb 12, 2026
11 checks passed
marcusquinn added a commit that referenced this pull request Feb 12, 2026
…1203)

Add escalating conflict resolution to rebase_sibling_pr():
1. Plain rebase (no conflicts)
2. Rebase with -Xtheirs (feature branch wins on conflicts)
3. AI CLI resolution for complex cases (Strategy 3 fallback)

Also adds:
- resolve_rebase_conflicts() for AI-assisted per-file resolution
- Stale worktree cleanup (abort stuck rebases, fix detached HEAD)
- Temp worktree creation when no worktree exists (avoids dirty tree)
- CONFLICTING added to t298 handler (was only BEHIND/DIRTY)
- Phase 7b: periodic retry of merge-conflict-blocked tasks (30min)

Tested: resolved 4 blocked PRs (#1171, #1187, #1188, #1191) that
were stuck with merge conflicts — all resolved with -Xtheirs.
@marcusquinn marcusquinn deleted the feature/t308 branch February 12, 2026 02:48
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.

1 participant