Skip to content

Maintain Open Session verification coverage - #384

Open
open-session-os-tella-dev[bot] wants to merge 1 commit into
mainfrom
auto-daily-verification-skill-maintenance-202609130600-os-01a0995a-3417-75cc-a230-07cc3e3df26a
Open

open-session-os-tella-dev[bot] wants to merge 1 commit into
mainfrom
auto-daily-verification-skill-maintenance-202609130600-os-01a0995a-3417-75cc-a230-07cc3e3df26a

Conversation

@open-session-os-tella-dev

Copy link
Copy Markdown
Contributor

Summary

  • pre-seed demo data before server startup and make doctor reject an incomplete demo session index
  • correct stale session, archive, and settings accessibility recipes
  • add the Databases feature introduced after the last verification map update

Verification

  • bun run check
  • full isolated source and live pass for sessions, automations, goals, archived sessions, settings, and databases
  • desktop 1440x900 and phone 390x844 evidence in artifacts/verification/opensession/verify-20260913-060358-3380440/

Started by Daily verification skill maintenance in this OS session

Co-authored-by: open-session-os-tella-dev[bot] <309495949+open-session-os-tella-dev[bot]@users.noreply.github.com>
@open-session-os-tella-dev

open-session-os-tella-dev Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor Author

🤖 OS review · request changes · quality 4/5 · risk low

Safe once the P2 below is fixed. Database verification is missing from the skill’s discovery metadata.

🟢 Risk low · recovery in minutes
A revert restores verification behavior; only disposable run state and prior evidence remain.

1 inline comment below.

Reviewed 3895e78 · GPT-5.6 Sol · open session · labels: os-auto-fix fix and push · os-adversarial deeper pass · os-simplify cleanup

🔁 Not merge-ready and no live session owns this branch — add the os-auto-fix label and I'll fix the findings automatically.

@vercel

vercel Bot commented Sep 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
opensession Ready Ready Preview Sep 13, 2026 6:17am UTC

@open-session-os-tella-dev open-session-os-tella-dev Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OS review · 3895e78

- [Goals](./goals.md) covers list and detail views, creating a goal, and confirming persistence.
- [Archived sessions](./archived-sessions.md) covers the archived index, search, filters, restoration, and empty results.
- [Settings](./settings.md) covers direct section routes, settings navigation, search, and persisted preferences or instance configuration.
- [Databases](./databases.md) covers creating and inspecting persistent SQLite stores, read-only queries, management, and phone navigation.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟠 P2 — Expose database coverage in the skill description

The new recipe is linked here, but .agents/skills/verify-opensession/SKILL.md still describes the skill as covering only “session, workspace, automation, goal, archive, or settings changes.” That frontmatter supplies the machine-facing skill catalog, so a request specifically to verify a Databases change may not select this skill and will never reach this map. Add database to the frontmatter description.

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.

0 participants