Adds Pulumi config to manage GitHub by cogni-ai-ee · Pull Request #1 · Cogni-AI-OU/github-git-ops

cogni-ai-ee · 2026-01-17T00:17:58Z

Summary by CodeRabbit

New Features
- Added Pulumi-based GitHub resource support to enable repo provisioning via IaC.
Documentation
- Added guides covering Pulumi-GitHub workflows, import/apply commands, state management, and Git history-rewrite guidance.
Chores
- Updated dev environment to include Pulumi tooling and dependency.
- Removed two devcontainer editor extensions.
- Updated ignore patterns to include Pulumi artifacts and related YAML files.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

coderabbitai · 2026-01-17T00:18:07Z

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Adds Pulumi infrastructure for managing GitHub repositories, updates devcontainer and Python requirements to include Pulumi, adjusts ignore rules to include Pulumi artifacts (and stop ignoring .token.json), and adds documentation and skill guides for Pulumi-GitHub and Git history-rewrite workflows.

Changes

Cohort / File(s)	Summary
Ignore & Lint Configs `\.cursorignore`, `\.gitignore`, `\.yamllint`	Stop ignoring `.token.json` in `.cursorignore`; add `Pulumi.*.yaml` and `.pulumi/` to ignore lists and lint ignores.
Devcontainer & Runtime deps `.devcontainer/devcontainer.json`, `.devcontainer/requirements.txt`	Remove two Claude VSCode extensions; add Pulumi devcontainer feature and add `pulumi-github` to devcontainer Python requirements.
Pulumi project `pulumi/Cogni-AI-OU/Pulumi.yaml`, `pulumi/Cogni-AI-OU/README.md`	New Pulumi YAML declaring three GitHub repositories (one repo + two templates), outputs for repository names, and pulumi tags; README with setup, import, and apply guidance.
Skill docs `.github/skills/pulumi-github/SKILL.md`, `.github/skills/git/SKILL.md`	New Pulumi-GitHub skill guide with usage/operational notes; Git skill extended with a "History Rewrites with Hooks" non-interactive workflow.

Sequence Diagram(s)

sequenceDiagram
  actor Developer as Developer
  participant Devcontainer as Devcontainer (IDE)
  participant PulumiCLI as Pulumi CLI
  participant GitHub as GitHub API
  participant State as Pulumi State

  rect rgba(135,206,250,0.5)
    Developer->>Devcontainer: open repo (devcontainer with Pulumi feature)
    Developer->>PulumiCLI: set env (GITHUB_TOKEN), configure stack
    Developer->>PulumiCLI: pulumi preview
    PulumiCLI->>GitHub: query resources (read)
    GitHub-->>PulumiCLI: current state
    PulumiCLI->>State: compare desired vs actual
    Note right of PulumiCLI: preview generated
    Developer->>PulumiCLI: pulumi up --yes
    PulumiCLI->>GitHub: create/update repository resources
    GitHub-->>PulumiCLI: confirm changes
    PulumiCLI->>State: persist new state
    PulumiCLI-->>Developer: apply complete / outputs
  end

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 I hopped through YAML, tags, and name,

Pulumi seeds sprouted into GitHub fame.
Devcontainer prepped, tokens set just right,
Templates and repos hopped into sight.
A tiny rabbit cheers this infra flight!

🚥 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 PR title 'Adds Pulumi config to manage GitHub' accurately reflects the main change: introducing Pulumi configuration to manage GitHub resources through Infrastructure as Code.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

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

coderabbitai

Actionable comments posted: 3

🤖 Fix all issues with AI agents

In `@pulumi/Cogni-AI-OU/Pulumi.yaml`:
- Around line 53-55: Pulumi repo config currently disables all merge strategies
(allowMergeCommit, allowRebaseMerge, allowSquashMerge); change Pulumi.yaml so at
least one of these keys is set to true (e.g., set allowSquashMerge: true) to
enable a permitted merge method and keep the others as desired. Ensure you
update the keys allowMergeCommit, allowRebaseMerge, or allowSquashMerge
accordingly in the Pulumi.yaml file so one merge strategy is enabled.
- Around line 32-34: The repository settings currently disable all merge
strategies (allowMergeCommit, allowRebaseMerge, allowSquashMerge); update
Pulumi.yaml to enable at least one merge method by setting one of these flags to
true (for example set allowSquashMerge: true) so the repo can accept merges;
edit the block containing allowMergeCommit, allowRebaseMerge, and
allowSquashMerge to reflect the chosen enabled option.
- Around line 9-11: The repository settings currently disable all merge
strategies (allowMergeCommit, allowRebaseMerge, allowSquashMerge all set to
false), preventing PRs from being merged; enable at least one
strategy—preferably set allowSquashMerge: true (or set allowMergeCommit: true /
allowRebaseMerge: true if you prefer) and keep the other flags as desired; apply
the same change to the other occurrences of these keys in the file where the
three flags are set (the other blocks with allowMergeCommit, allowRebaseMerge,
allowSquashMerge) so at least one merge strategy is enabled across all
repository blocks.

🧹 Nitpick comments (1)

.devcontainer/requirements.txt (1)

13-13: Consider pinning pulumi-github version for reproducibility.

The pulumi-github package is added without version pinning. While this matches the pattern of most other dependencies in this file, unpinned dependencies can lead to unexpected breaking changes or inconsistent environments. Consider pinning to a specific version (e.g., pulumi-github>=6.0.0) to ensure reproducible builds.

coderabbitai · 2026-01-17T00:22:57Z

+      allowMergeCommit: false
+      allowRebaseMerge: false
+      allowSquashMerge: false


⚠️ Potential issue | 🔴 Critical

All merge strategies are disabled—PRs cannot be merged.

Setting allowMergeCommit, allowRebaseMerge, and allowSquashMerge all to false prevents any pull request from being merged via the GitHub UI. At least one merge strategy must be enabled.

🐛 Proposed fix to enable squash merge (recommended for cleaner history)

allowMergeCommit: false allowRebaseMerge: false - allowSquashMerge: false + allowSquashMerge: true

Apply the same fix to the other repositories at lines 32-34 and 53-55.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

allowMergeCommit: false

allowRebaseMerge: false

allowSquashMerge: false

allowMergeCommit: false

allowRebaseMerge: false

allowSquashMerge: true

🤖 Prompt for AI Agents

In `@pulumi/Cogni-AI-OU/Pulumi.yaml` around lines 9 - 11, The repository settings currently disable all merge strategies (allowMergeCommit, allowRebaseMerge, allowSquashMerge all set to false), preventing PRs from being merged; enable at least one strategy—preferably set allowSquashMerge: true (or set allowMergeCommit: true / allowRebaseMerge: true if you prefer) and keep the other flags as desired; apply the same change to the other occurrences of these keys in the file where the three flags are set (the other blocks with allowMergeCommit, allowRebaseMerge, allowSquashMerge) so at least one merge strategy is enabled across all repository blocks.

coderabbitai · 2026-01-17T00:22:57Z

+      allowMergeCommit: false
+      allowRebaseMerge: false
+      allowSquashMerge: false


⚠️ Potential issue | 🔴 Critical

Same merge strategy issue—all disabled.

Same issue as repo_github_git_ops: no merge method is enabled. Enable at least one.

🤖 Prompt for AI Agents

In `@pulumi/Cogni-AI-OU/Pulumi.yaml` around lines 32 - 34, The repository settings currently disable all merge strategies (allowMergeCommit, allowRebaseMerge, allowSquashMerge); update Pulumi.yaml to enable at least one merge method by setting one of these flags to true (for example set allowSquashMerge: true) so the repo can accept merges; edit the block containing allowMergeCommit, allowRebaseMerge, and allowSquashMerge to reflect the chosen enabled option.

coderabbitai · 2026-01-17T00:22:57Z

+      allowMergeCommit: false
+      allowRebaseMerge: false
+      allowSquashMerge: false


⚠️ Potential issue | 🔴 Critical

Same merge strategy issue—all disabled.

Same issue as the other repositories: no merge method is enabled. Enable at least one.

🤖 Prompt for AI Agents

In `@pulumi/Cogni-AI-OU/Pulumi.yaml` around lines 53 - 55, Pulumi repo config currently disables all merge strategies (allowMergeCommit, allowRebaseMerge, allowSquashMerge); change Pulumi.yaml so at least one of these keys is set to true (e.g., set allowSquashMerge: true) to enable a permitted merge method and keep the others as desired. Ensure you update the keys allowMergeCommit, allowRebaseMerge, or allowSquashMerge accordingly in the Pulumi.yaml file so one merge strategy is enabled.

cogni-ai-ee added 6 commits January 17, 2026 00:21

Remove unused VS Code extensions from devcontainer configuration

937e6ff

Add Pulumi feature to devcontainer and update requirements

7067b0e

Add Pulumi configuration files and update .gitignore

8bc8685

Add Pulumi GitHub skill documentation for resource management

2b4265d

Update .cursorignore to include Pulumi YAML files

db3922f

Add troubleshooting references and links to README.md

e4f98d1

cogni-ai-ee force-pushed the dev branch from 493ed01 to e4f98d1 Compare January 17, 2026 00:21

coderabbitai Bot reviewed Jan 17, 2026

View reviewed changes

Add history rewrite techniques with hooks to SKILL.md

32302d7

cogni-ai-ee merged commit 2c990a1 into main Jan 17, 2026
6 of 7 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Adds Pulumi config to manage GitHub#1

Adds Pulumi config to manage GitHub#1
cogni-ai-ee merged 7 commits into
mainfrom
dev

cogni-ai-ee commented Jan 17, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Jan 17, 2026 •

edited

Loading

Review failed

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Poem

Uh oh!

coderabbitai Bot left a comment

Uh oh!

coderabbitai Bot Jan 17, 2026

Uh oh!

coderabbitai Bot Jan 17, 2026

Uh oh!

coderabbitai Bot Jan 17, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

cogni-ai-ee commented Jan 17, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Jan 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review failed

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Poem

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jan 17, 2026

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jan 17, 2026

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jan 17, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

cogni-ai-ee commented Jan 17, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Jan 17, 2026 •

edited

Loading