This repository was archived by the owner on Nov 3, 2026. It is now read-only.
Repository navigation
refactor!: meta_cli reads only .meta.yaml - #47
Merged
Merged
Conversation
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]]
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
…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]]
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The not-found message names
.meta.yaml. The orphan hint prints only the YAML form.meta init claudeno longer refuses a.metafile (it is no longer a config), and the plugin code no longer readsNotADirectoryunder.meta/pluginsas "no plugins" (that existed only for a.metaconfig file). Format and legacy tests are deleted. New:old_config_names_are_not_a_workspaceanda_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 saysgit worktree removeworks 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]]