Skills

Feature Workflow

Taking a multi-PR feature from "shape unknown" to merged without five review rounds per...

Install

npx skills add apache/tika --skill feature-workflow -g -a claude-code -y
Repository
apache/tika4.1k
Category
Git and review
Updated
3d ago
License
Apache-2.0
Context cost
Lightabout 1,078 tokens
Runs scripts
No, instructions only

What it does

Taking a multi-PR feature from "shape unknown" to merged without five review rounds per PR: spike until interfaces stop moving, write the contract, cut PRs along contract seams, one review per PR. Use when starting a feature that touches more than one lifecycle object or public interface, when a PR review keeps changing interfaces, or when splitting a large branch.

<!-- Licensed to the Apache Software Foundation (ASF) under one or more contributor license agreements. See the NOTICE file distributed with this work for additional information regarding copyright ownership. The ASF licenses this file to You under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0 Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License. -->

Local override: $TIKA_SKILLS_LOCAL/feature-workflow/LOCAL.md (default ~/.tika-skills), read after this file, wins on conflict.

Feature Workflow: spike, contract, cut, ship

A complex feature's shape is learned by building it. Learning it on the PR costs a review round per lesson and reshapes what the next round reviews. Keep learning and shipping on different branches.

1. Spike

Throwaway branch. Build end to end, roughly: no CHANGES, docs, or polish; change neighbors freely. Review it (.skills/devs/pr-review/SKILL.md) and let findings reshape interfaces.

Exit: the last review changed edge-case handling, not an interface. While reviews still rename, split, or add methods, keep spiking (pr-review verdict "still spiking").

2. Contract

Write down what the spike taught, half a page per lifecycle object (open/close, acquire/release, publish/abort, spill, rewind): states and transitions, each method's behavior per state, resource ownership on success and every failure path, threading. Put it in the type's javadoc.

Read the whole SKILL.md on GitHub

More in Git and review