Repository navigation
chore/tidy-repo-files - #60
Merged
Merged
Conversation
Second pass over AI_RULES.md section 7. The circadian source image is renamed to source.png (content unchanged) with .homeyignore updated so it stays out of the app bundle. robots.md loses its assistant credit line, DOCUMENTATION_PROMPT_TEMPLATE.md no longer names the conventions file, the .gitignore comment is neutral, and WORKLOG.md no longer repeats the names the first pass removed. A new WORKLOG.md entry logs this pass, including two signed PR comments that were edited on GitHub.
SESSION_SUMMARY.md and the generated reports in no.tiwas.booleantoolbox/coverage/ are not referenced anywhere. coverage/ is added to .gitignore so test:coverage output is not committed again. WORKLOG.md records the removal and the deleted branches.
New workflow .github/workflows/message-check.yml runs .github/scripts/message-check.js on pull requests, pushes, issues, comments and reviews. It removes signature footers from PR descriptions and comments, and fails on branch names, commit messages, identities and PR text that break the message rules in AI_RULES.md section 7. Tests run with node --test .github/scripts/message-check.test.js. AI_RULES.md names the check, PROJECT_DOCUMENTATION.md lists it and WORKLOG.md records it.
Tiwas
changed the title
Tiwas
changed the title
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Repo housekeeping, plus a new check for commit messages, branch names and PR text.
Changes
.github/workflows/message-check.ymlruns.github/scripts/message-check.json pull requests, pushes, issues, comments and reviews. It applies the message rules inAI_RULES.md:source.png(content unchanged), matching the other source images..homeyignoreupdated so it stays out of the app bundle.robots.mdandDOCUMENTATION_PROMPT_TEMPLATE.md: outdated lines removed..gitignore: comment reworded, andcoverage/added.SESSION_SUMMARY.md(old session note) and the 27 generated coverage reports inno.tiwas.booleantoolbox/coverage/. Nothing references them. They account for almost all of the ~18,000 deleted lines.AI_RULES.md,PROJECT_DOCUMENTATION.mdandWORKLOG.mdupdated.No app code, manifest or config schema changed, so no companion tool is affected.
Verification
npx jest --runInBand: 23 suites, 358 tests passed.npm run test:package: publish validation passed (910 files, 8.00 MB, 17 manifest assets verified).node --test .github/scripts/message-check.test.js: 23 tests passed.mainwith no false hits, and end to end with Actions-style events against a fake API. Its first real run on GitHub is on this PR.