Skip to content

feat(aidd-user-testing): prefer PageSpace DOCUMENT pages for test scripts#216

Open
2witstudios wants to merge 1 commit into
mainfrom
feat/aidd-user-testing-pagespace
Open

feat(aidd-user-testing): prefer PageSpace DOCUMENT pages for test scripts#216
2witstudios wants to merge 1 commit into
mainfrom
feat/aidd-user-testing-pagespace

Conversation

@2witstudios

Copy link
Copy Markdown

Summary

  • Human and agent test scripts are now saved as DOCUMENT pages under a "User Testing" folder in the project's PageSpace drive when mcp__pagespace__* tools are connected.
  • Falls back to $projectRoot/plan/ when PageSpace is unavailable — non-breaking.

Why

Test scripts stored as PageSpace DOCUMENT pages are visible and shareable in the workspace alongside the user journeys and personas from aidd-product-manager. Keeping the full testing workflow in one place makes it easier to iterate on scripts across sessions.

Changes

  • ai/skills/aidd-user-testing/SKILL.mdFileLocations section updated with PageSpace-first, filesystem-fallback branching

Human and agent test scripts are now saved as DOCUMENT pages under a
"User Testing" folder in the project's PageSpace drive when MCP tools
are connected. Falls back to $projectRoot/plan/ when unavailable.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7bf6315111

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +123 to +125
Pagespace available => save test scripts as DOCUMENT pages in the project's Pagespace drive (preferred)
Human and agent scripts as child pages under a "User Testing" folder
No Pagespace => save to $projectRoot/plan/ folder (create if not present):

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Update the interface to preserve PageSpace-first storage

When PageSpace is available, this new instruction conflicts with the later /user-test interface at line 143, which still explicitly directs the agent to save scripts to $projectRoot/plan/; ai/skills/aidd-user-testing/README.md:17-19 repeats the filesystem-only behavior. Because this skill is itself the executable prompt, agents can follow the more specific later instruction and bypass PageSpace entirely, making the advertised PageSpace-first behavior unreliable. Update the interface and command-facing documentation to describe the same PageSpace-first fallback.

Useful? React with 👍 / 👎.

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.

1 participant