Skip to content

feat: ontrack.yml metadata tracking for reporting directories#23

Merged
dimalvovs merged 6 commits intomainfrom
copilot/create-metadata-tracking-feature
Apr 13, 2026
Merged

feat: ontrack.yml metadata tracking for reporting directories#23
dimalvovs merged 6 commits intomainfrom
copilot/create-metadata-tracking-feature

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 13, 2026

  • Replace text with track as required metadata field; add config-backed validation
  • Remove owner and created from _REQUIRED_METADATA_FIELDS (keep only "track")
  • Update _is_on_track docstring
  • Update affected tests (112 passing)
  • Update README and ontrack.yml comments

Copilot AI and others added 2 commits April 13, 2026 15:46
- Add _load_ontrack_yml, _get_directory_metadata, _is_on_track helpers
- Modify _find_reporting_directories: ontrack.yml stops descent and makes
  all non-ignored sibling subdirectories reporting directories; ontrack.yml
  itself is never treated as a visible file
- Modify _build_directory_entry to include on_track bool and metadata dict
- Modify report_directory to print On track / metadata fields
- Add 55 new tests covering all new functions and edge cases
- Update README with ontrack.yml format, required fields, and output examples

Agent-Logs-Url: https://github.com/FertigLab/ontrack/sessions/163c3f34-6652-4173-b6d1-ff1d54993092

Co-authored-by: dimalvovs <1246862+dimalvovs@users.noreply.github.com>
…in report_directory

Agent-Logs-Url: https://github.com/FertigLab/ontrack/sessions/163c3f34-6652-4173-b6d1-ff1d54993092

Co-authored-by: dimalvovs <1246862+dimalvovs@users.noreply.github.com>
@dimalvovs
Copy link
Copy Markdown
Contributor

@copilot please add an example ontrack.yml file

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 13, 2026

@copilot please add an example ontrack.yml file

Added ontrack.yml to the repository root in commit 4e1e7d1. It demonstrates required fields (description, owner, created) and optional extras (pi, grant, status) across three example projects.

Copilot AI requested a review from dimalvovs April 13, 2026 16:41
@dimalvovs
Copy link
Copy Markdown
Contributor

@copilot change the "Description" mandatory meta field name to "Text" for brevity.

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 13, 2026

@copilot change the "Description" mandatory meta field name to "Text" for brevity.

Done in commit ecd3347. The required metadata field description has been renamed to text across ontrack.py (_REQUIRED_METADATA_FIELDS), the example ontrack.yml, README.md, and all tests. All 106 tests still pass.

…e against config.yaml track section

Agent-Logs-Url: https://github.com/FertigLab/ontrack/sessions/217d8196-2bc0-4fcc-ba1e-6f6b98520644

Co-authored-by: dimalvovs <1246862+dimalvovs@users.noreply.github.com>
…d are now optional

Agent-Logs-Url: https://github.com/FertigLab/ontrack/sessions/650dfd5a-723a-45ff-a585-5594bc368a5f

Co-authored-by: dimalvovs <1246862+dimalvovs@users.noreply.github.com>
@dimalvovs dimalvovs marked this pull request as ready for review April 13, 2026 19:14
@dimalvovs dimalvovs merged commit d708afe into main Apr 13, 2026
1 check passed
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