README: - Quick Start shows kappa add workflow instead of manual configs - New features: indexed repos, env operators (+=, ?=), conflict reporting - Updated subcommand table (add, remove, index, validate-index) - Repo maintenance section (kappa index) - Link to STYLEGUIDE.md, test counts, CI info CONTRIBUTING: - Updated test counts (30+23+85 instead of 31+23) - Added test-full.sh to mandatory test suite - Referenced STYLEGUIDE.md in PR requirements - Updated good-first/ambitious issues for current state
5.0 KiB
Contributing to kappa
We're building a package manager that doesn't care about your init system, your bootloader, or your life choices. If that sounds like your kind of project, keep reading.
The rules
These aren't guidelines. They're the deal.
1. C++23 or don't bother
We compile with Clang, -std=c++23, and zero warnings. If your code needs a
polyfill for std::format or can't handle designated initializers, it
doesn't belong here. The standard library is your only dependency. Zero
external C++ libraries. Not even Boost.
// ✓ yes
auto msg = std::format("building {} (depth={})", name, depth);
// ✗ no
auto msg = fmt::format("building {} (depth={})", name, depth);
2. Every new module mirrors the existing structure
include/kappa/{module}/
├── types.hpp # enums, structs, parse/validate declarations
├── {feature}.hpp # public interface
src/{module}/
├── types.cpp # implementations
├── backend_a.cpp # per-variant generators
├── backend_b.cpp
└── install.cpp # dispatch + orchestration
If you're adding a feature, look at src/service/ or src/boot/ for the
pattern. If your new module doesn't look like those, you're doing it wrong.
3. Never suppress type safety
There is no as any, no @ts-ignore, no reinterpret_cast abuse, and
no void* unless you're talking to the kernel. If the type system is
fighting you, you're fighting the design. Fix the design.
4. Use the namespace. All of it.
namespace kappa::module {
// everything goes here
} // namespace kappa::module
No using namespace std; at file scope. No anonymous namespaces for
functions that are used across files — extract to util.hpp instead.
5. Thread safety is not optional
The scheduler is multithreaded. If you touch shared state, you own the
lock. std::mutex, std::atomic, std::condition_variable — use them
correctly or don't use them at all. If you don't know what memory_order
means, stay out of the scheduler.
6. Tests are shell scripts.
Integration tests live in three suites. If you add a subcommand, add a test.
./test.sh # 30 tests — DSL parsing, config, services, init systems
./test-init-switch.sh # 23 tests — init-switching, rebuild impact, bootloaders
./test-full.sh # 85 tests — CLI, all keywords, repos, add/remove,
# resolver conflicts, deptrees, build, env ops,
# index generation, --root, imports, doctor, and more
All three must pass. CI enforces this on every push to main.
7. Backward compatibility is mandatory
The .kap DSL is the contract. You can add keywords. You cannot remove
them. You can extend syntax. You cannot break existing configs. If your
change means someone's config.kap stops parsing, it doesn't ship.
How to contribute
Pick something
Good first issues:
- Adding a 6th init system backend (we have 5: systemd, openrc, s6, runit, dinit)
- Adding a 3rd bootloader backend (we have 2: grub, limine)
- Improving the
kappa doctordiagnostics for package recipes - Adding
--features/--configflags tokappa add - Shell completion scripts (bash, zsh, fish)
Ambitious issues:
- Binary package support (pre-built caches)
- Remote build farm (distcc-style)
- Signed package verification with index signing
- Filesystem overlay activation (like Nix profiles)
- Transitive dependency resolution (auto-including deps not in config)
Send a PR
- Fork the repo
- Create a branch:
feat/my-thingorfix/my-bug - Write code that follows the rules above
- Run
./test.sh && ./test-init-switch.sh && ./test-full.sh— all three must pass - Open a PR against
main
PR requirements
- Build must pass:
cmake --build buildwith zero warnings on Clang 17+ - Tests must pass: all three shell test suites
- Follow the conventions in STYLEGUIDE.md
- No commented-out code. No dead code. No TODO without a date.
- Commit messages in imperative:
Add runit backendnotAdded runit backend
What we won't merge
systemd-only features. If it can't work on at least two init systems, it goes in asystemdpackage definition, not in kappa.- Dependency on a specific distro. Kappa runs on any Linux kernel. No
hardcoded paths to
/usr/lib/systemd, no assumptions about/etc/os-release. - Abstract nonsense. FactoryFactoryBuilder patterns. Premature generalization. If you need three layers of indirection to add a feature, the feature is too complicated.
- AI slop. If it looks like ChatGPT wrote it, it gets rejected. We can tell. Write code like a human who's been doing this for a decade.
Communication
We don't have a Discord. We don't have a forum. Open an issue. Write a clear title, a reproduction case, and what you expected. We'll respond when we respond.
If you want to propose a major feature, open an issue first. Surprise PRs that rewrite half the codebase get closed without review.
Kappa is 0.1.0. Everything is subject to change except the rules above.