Skip to content
This repository was archived by the owner on Aug 23, 2026. It is now read-only.

Show current Campus work and GitHub Issue - #36

Merged
pirajoke merged 2 commits into
mainfrom
codex/issue-34-campus-project-issue
Aug 18, 2026
Merged

Show current Campus work and GitHub Issue#36
pirajoke merged 2 commits into
mainfrom
codex/issue-34-campus-project-issue

Conversation

@pirajoke

Copy link
Copy Markdown
Owner

Closes #34

Summary

  • show a concise current-work summary in the read-only Campus project inspector
  • add a keyboard-accessible GitHub Issue link for validated owner-view tasks
  • keep anonymous public responses free of owner-only fields and raw task content

Safety

  • summary comes only from typed objective metadata
  • Issue identity is validated against an exact HTTPS github.com issue URL
  • malformed or unsafe optional fields fail closed and hide only their own row
  • no dispatch, edit, merge, deploy, or runtime mutation

Verification

  • full builder test suite: 172 passed
  • Python compile check: passed
  • JavaScript syntax check: passed
  • diff check: passed
  • independent privacy/security/accessibility review: GO

Note

Live browser rendering was unavailable in this session; responsive and DOM behavior is covered by deterministic Campus tests and static review.

@pirajoke
pirajoke marked this pull request as ready for review August 18, 2026 06:11
@pirajoke
pirajoke merged commit 4f9e271 into main Aug 18, 2026
1 check passed

@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: 0cfea2a9bd

ℹ️ 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 +1421 to +1427
write_method = getattr(self, "command", "").upper() in {
"POST",
"PUT",
"PATCH",
"DELETE",
}
expected = _dashboard_run_token(create_if_missing=write_method)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Provision the owner token before authenticated writes

On a fresh deployment or after the token file is removed, every GET now calls _dashboard_run_token(create_if_missing=False), while the deployment script never creates the file and mm-command-center-auth immediately tries to read it. The public UI also disables its write action until a token is supplied, so the documented authorization flow cannot bootstrap without first issuing an undocumented unauthenticated POST solely to create the token; provision it during deployment/startup instead.

Useful? React with 👍 / 👎.

Comment on lines +118 to +121
const token = window.localStorage
.getItem('command-center.jarvis-run-token')
?.trim();
if (token) headers['X-Dashboard-Run-Token'] = token;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Forward the token into the cross-origin Campus iframe

When the supported dashboard is opened from file://, PIXEL_AGENTS_BASE loads this script in an iframe from https://command.meshly.fr, but the saved run token belongs to the parent file origin. Reading window.localStorage here therefore cannot see that token, so every Campus refresh receives the anonymous projection and the new summary/Issue rows remain unavailable to the owner; explicitly pass the token from the parent to the iframe or perform the authenticated request in the parent.

Useful? React with 👍 / 👎.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show current work summary and GitHub Issue in Campus project details

1 participant