Skip to content
This repository was archived by the owner on Nov 3, 2026. It is now read-only.

refactor!: meta_cli reads only .meta.yaml - #47

Merged
gold-silver-copper merged 5 commits into
mainfrom
feat/meta-yaml-only
Oct 28, 2026
Merged

gold-silver-copper merged 5 commits into
mainfrom
feat/meta-yaml-only

Conversation

@gold-silver-copper

@gold-silver-copper gold-silver-copper commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

The not-found message names .meta.yaml. The orphan hint prints only the YAML form. meta init claude no longer refuses a .meta file (it is no longer a config), and the plugin code no longer reads NotADirectory under .meta/plugins as "no plugins" (that existed only for a .meta config file). Format and legacy tests are deleted. New: old_config_names_are_not_a_workspace and a_meta_file_where_the_folder_goes_is_not_taken_for_no_plugins. The six skills name only .meta.yaml. The meta-worktree skill no longer says git worktree remove works for a member whose repo went with its originating checkout (CodeRabbit on meta#128).

Part of gitkb/meta#132 (meta reads only .meta.yaml), which has the reasoning, the breaking-change note and the verification. Merge together with it.

Task: [[tasks/harmony-2878]]

meta discovers its config only as .meta.yaml (and, until Matt confirms its removal, .meta.yml). The .meta and .meta.json names, the JSON parse path and ConfigFormat are gone: every config is parsed as YAML, which reads JSON content too. A folder with only .meta or .meta.json is not a workspace.

BREAKING CHANGE: rename a .meta or .meta.json config to .meta.yaml (same content).

[[tasks/harmony-2878]]
Matt approved dropping .meta and .meta.json; .meta.yml was not in his note, so it goes in its own commit to keep or drop. With one name a folder can no longer hold two configs, so the "Multiple meta config files" error and its tests go too.

BREAKING CHANGE: rename a .meta.yml config to .meta.yaml.

[[tasks/harmony-2878]]
@coderabbitai

coderabbitai Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: da62d1d5-7b1b-43f4-a51f-e7fc085e0fd0
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

…examples

Review round 1. With the NotADirectory exemption gone, a stray .meta file gets the plugins-directory warning and plugin list --local fails, instead of both quietly reading it as no plugins. tests/examples/.meta and .meta2 were unused old-name fixtures.

[[tasks/harmony-2878]]
Review round 2. The meta-workspace skill description said .meta/.meta.yaml; the meta-worktree skill says to copy out files before deleting a member git can no longer remove; the plugin workspace error test checks it names .meta.yaml.

[[tasks/harmony-2878]]
Review round 3.

[[tasks/harmony-2878]]
@gold-silver-copper
gold-silver-copper merged commit c4ce6b1 into main Oct 28, 2026
7 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant