Skip to content

Wire private AI skills submodule (.dspace-skills)#1343

Merged
milanmajchrak merged 1 commit into
dtq-devfrom
add_dspace-skills_submodule
Jun 25, 2026
Merged

Wire private AI skills submodule (.dspace-skills)#1343
milanmajchrak merged 1 commit into
dtq-devfrom
add_dspace-skills_submodule

Conversation

@jr-rk

@jr-rk jr-rk commented Jun 24, 2026

Copy link
Copy Markdown

Problem description

Wire private .dspace-skills submodule to DSpace app.

Analysis

(Write here, if there is needed describe some specific problem. Erase it, when it is not needed.)

Problems

(Write here, if some unexpected problems occur during solving issues. Erase it, when it is not needed.)

Copilot review

  • Requested review from Copilot

Summary by CodeRabbit

  • Documentation
    • Added and updated repository guidance for contributors and automation.
    • Clarified how to initialize and use the included submodule when available.
    • Added instructions for choosing the appropriate project profile and starting tasks from the shared skills list.
    • Recorded the latest submodule reference so the repository points to the expected version.

@jr-rk jr-rk requested a review from Copilot June 24, 2026 15:43
@jr-rk jr-rk self-assigned this Jun 24, 2026
@jr-rk jr-rk changed the title chore: wire private AI skills submodule (.dspace-skills) Wire private AI skills submodule (.dspace-skills) Jun 24, 2026
@coderabbitai

coderabbitai Bot commented Jun 24, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 99568ea5-4353-4e99-886e-8cbf374d8b30

📥 Commits

Reviewing files that changed from the base of the PR and between eebe17d and 62721e4.

📒 Files selected for processing (4)
  • .dspace-skills
  • .gitmodules
  • AGENTS.md
  • CLAUDE.md

📝 Walkthrough

Walkthrough

Adds a .dspace-skills git submodule to the repository, registered in .gitmodules with a URL and main branch pin, and pointed to a specific commit. Two new agent instruction files (AGENTS.md, CLAUDE.md) are added with identical guidance for conditionally loading private agent profiles and skills from the submodule.

Changes

dspace-skills Submodule and Agent Guidance

Layer / File(s) Summary
Submodule registration and commit pin
.gitmodules, .dspace-skills
Registers the .dspace-skills submodule with its path, remote URL, and main branch in .gitmodules, and records the pinned subproject commit hash.
Agent instruction files for submodule usage
AGENTS.md, CLAUDE.md
Adds both agent guidance files with conditional logic: when .dspace-skills/ is present, use its AGENTS.md as authoritative, load the appropriate frontend/backend profile, pull skills on demand from .dspace-skills/skills/, and start tasks from .dspace-skills/SKILLS.md; includes fallback behavior and the submodule init command.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly states the main change: wiring the private .dspace-skills submodule.
Description check ✅ Passed The description includes the problem statement and Copilot review, though Analysis and Problems are left as placeholders.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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


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.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds repository wiring/documentation for an optional private .dspace-skills/ git submodule intended to provide internal team AI playbooks, while allowing outside contributors to ignore it when unavailable.

Changes:

  • Add CLAUDE.md and AGENTS.md guidance pointing to .dspace-skills/* documentation when the submodule is present.
  • Add .gitmodules entry defining the .dspace-skills submodule repository and branch.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
CLAUDE.md Documents how to use the private .dspace-skills/ submodule for agent guidance when available.
AGENTS.md Mirrors the same guidance for agent tooling that reads AGENTS.md.
.gitmodules Declares the .dspace-skills submodule configuration (path/url/branch).

Comment thread .gitmodules
@jr-rk jr-rk requested a review from milanmajchrak June 24, 2026 15:47
@milanmajchrak milanmajchrak merged commit ee37116 into dtq-dev Jun 25, 2026
16 checks passed
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.

3 participants