Skip to content

Conversation

@gotalab
Copy link
Owner

@gotalab gotalab commented Dec 6, 2025

Summary

  • Add ~/.codex/skills (Codex) as a selectable option during skillport init
  • Add descriptive labels to clarify which agent uses each path (e.g., Claude Code, Codex)

Test plan

  • uv run ruff check src/ passes
  • uv run pytest passes (362 tests)
  • uv run verify_server.py passes
  • Manual verification of init command shows new options

🤖 Generated with Claude Code

gotalab and others added 2 commits December 6, 2025 19:35
Add ~/.codex/skills as a selectable option during skillport init.
Also add descriptive labels to clarify which agent uses each path.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Copilot AI review requested due to automatic review settings December 6, 2025 10:36
@gotalab gotalab merged commit 9d44f6e into main Dec 6, 2025
7 checks passed
@gotalab gotalab deleted the fix/init-improvements branch December 6, 2025 10:37
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for Codex skills directory during initialization and applies consistent code formatting across the codebase.

  • Adds ~/.codex/skills (Codex) as a selectable option in the skillport init command
  • Adds descriptive agent labels to existing directory options (e.g., .claude/skills (Claude Code))
  • Applies consistent code formatting (line breaks, alignment) across multiple files

Reviewed changes

Copilot reviewed 32 out of 33 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/skillport/interfaces/cli/commands/init.py Adds Codex skills directory option and agent labels to init command choices
src/skillport/shared/types.py Reformats Field definitions for improved readability
src/skillport/shared/filters.py Reformats list comprehension to single line
src/skillport/shared/exceptions.py Reformats error message string to single line
src/skillport/shared/config.py Reformats Field definitions and error messages for consistency
src/skillport/modules/skills/public/update.py Reformats function signatures and conditionals, removes extra blank line
src/skillport/modules/skills/public/types.py Reformats Field definitions for multiline consistency
src/skillport/modules/skills/public/read.py Expands TEXT_EXTENSIONS set to multiple lines for readability
src/skillport/modules/skills/public/load.py Reformats conditional expressions to single lines
src/skillport/modules/skills/public/list.py Reformats conditional expression to single line
src/skillport/modules/skills/public/add.py Reformats complex expressions and conditionals to multiple lines
src/skillport/modules/skills/internal/validation.py Reformats ValidationIssue instantiation to single line
src/skillport/modules/skills/internal/manager.py Reformats error messages and return statements to single lines
src/skillport/modules/skills/internal/github.py Reformats function signatures, adds spacing for slice notation consistency
src/skillport/modules/indexing/public/query.py Removes extra blank line
src/skillport/modules/indexing/public/index.py Reformats return statements to single lines
src/skillport/modules/indexing/internal/state.py Reformats function signatures and conditionals
src/skillport/modules/indexing/internal/search_service.py Reformats function signatures and error messages
src/skillport/modules/indexing/internal/models.py Removes leading blank line
src/skillport/modules/indexing/internal/lancedb.py Reformats comprehensions, conditionals, and function calls
src/skillport/modules/indexing/internal/embeddings.py Reformats function calls to single lines
src/skillport/interfaces/mcp/instructions.py Reformats method chain to single line
src/skillport/interfaces/cli/theme.py Reformats Theme dictionary and function signature
src/skillport/interfaces/cli/commands/update.py Reformats conditionals and list operations
src/skillport/interfaces/cli/commands/show.py Reformats Panel instantiation to multiple lines
src/skillport/interfaces/cli/commands/serve.py Removes extra blank line
src/skillport/interfaces/cli/commands/remove.py Reformats console.print_json calls to multiple lines
src/skillport/interfaces/cli/commands/list.py Reformats long console.print statement
src/skillport/interfaces/cli/commands/lint.py Reformats list comprehension, console.print_json, and Panel calls
src/skillport/interfaces/cli/commands/doc.py Reformats long string to single line
src/skillport/interfaces/cli/commands/add.py Reformats complex expressions and console output calls
src/skillport/interfaces/cli/auto_index.py Reformats console.print statement to single line
src/skillport/interfaces/cli/app.py Reformats multiline help strings for command definitions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants