Skip to content

Mobile-first rewrite of index.html, add static smoke test, update Pages workflow and README#7

Open
Huỳnh Thương (Huynhthuongg) wants to merge 6 commits into
mainfrom
codex/review-code-and-propose-fixes
Open

Mobile-first rewrite of index.html, add static smoke test, update Pages workflow and README#7
Huỳnh Thương (Huynhthuongg) wants to merge 6 commits into
mainfrom
codex/review-code-and-propose-fixes

Conversation

@Huynhthuongg

@Huynhthuongg Huỳnh Thương (Huynhthuongg) commented Jun 4, 2026

Copy link
Copy Markdown
Member

Motivation

  • Modernize the single-file web app to a mobile-first UI with refreshed styling and Vietnamese localization.
  • Prevent bad static deploys by adding a lightweight smoke test that validates critical routes, chat controls, and inline script syntax before uploading the Pages artifact.
  • Clarify README and ensure the GitHub Pages workflow runs the smoke test as part of the build.
  • Fix a small comment typo in the Hadolint workflow.

Description

  • Overhauled index.html with a mobile-first layout, new CSS tokens, SVG icon set, simplified route set, and Vietnamese copy for UI text and labels.
  • Added scripts/smoke-test-static.mjs, an automated Node smoke test that asserts presence of <!DOCTYPE html>, section containers for home and ai, id="chatInput" and id="sendChat", checks message escaping usage, extracts inline scripts and runs a Node syntax check.
  • Updated .github/workflows/static.yml to run the smoke test (node scripts/smoke-test-static.mjs) during the build job before preparing the _site artifact.
  • Updated README.md to document the new smoke test, adjust the Pages workflow steps, add the smoke-test script to the project tree, and make small wording/roadmap adjustments.
  • Fixed a typo in the comment of .github/workflows/hadolint.yml (hadolint spelling).

Testing

  • Ran the static smoke test locally with node scripts/smoke-test-static.mjs, which completed and reported success.
  • Integrated the smoke test into the GitHub Actions build job in .github/workflows/static.yml so the test will run automatically on repository push and workflow_dispatch events.
  • No other automated tests were modified or executed as part of this change.

Codex Task


Summary by cubic

Mobile‑first rewrite of index.html with Vietnamese UI, hash routes, and a bottom‑nav + drawer + desktop rail. Adds a static smoke test to the Pages workflow, hardens chat rendering and mobile overflow, and updates CNAME to app.rkix.com.

  • New Features

    • Rebuilt index.html with a mobile‑first layout, line‑SVG icons, updated tokens, and VN copy; routes: home, ai, workspace, code, profile.
    • Added scripts/smoke-test-static.mjs and wired it in .github/workflows/static.yml; it checks required routes, chat controls, escaped message rendering, inline script syntax, and an overflow guard. Updated README.md with usage.
  • Bug Fixes

    • Escaped chat messages to prevent XSS; enforced overflow-x: clip/hidden to stop mobile overflow.
    • Corrected SLSA publish output to steps.hash.outputs.hashes; fixed kustomize setup in .github/workflows/google.yml; corrected a typo in .github/workflows/hadolint.yml.

Written for commit 45b6bbf. Summary will update on new commits.

Review in cubic

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@coderabbitai

coderabbitai Bot commented Jun 4, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@Huynhthuongg, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 58 minutes and 57 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: efd6d56a-a20e-408b-8199-5b4235961bc8

📥 Commits

Reviewing files that changed from the base of the PR and between 02cce37 and 45b6bbf.

📒 Files selected for processing (8)
  • .github/workflows/generator-generic-ossf-slsa3-publish.yml
  • .github/workflows/google.yml
  • .github/workflows/hadolint.yml
  • .github/workflows/static.yml
  • CNAME
  • README.md
  • index.html
  • scripts/smoke-test-static.mjs
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/review-code-and-propose-fixes
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch codex/review-code-and-propose-fixes
  • 🛠️ Aethon

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@kilo-code-bot

kilo-code-bot Bot commented Jun 4, 2026

Copy link
Copy Markdown

Kilo Code Review could not run — your account is out of credits.

Add credits or switch to a free model to enable reviews on this change.

@Huynhthuongg Huỳnh Thương (Huynhthuongg) added bug Something isn't working documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers invalid This doesn't seem right question Further information is requested labels Jun 4, 2026

@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: 8af84b424a

ℹ️ 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 (@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 (@codex) address that feedback".

Comment thread index.html Outdated
Comment thread index.html Outdated
@github-project-automation github-project-automation Bot moved this from Ready to In progress in @Huynhthuongg's untitled project Jun 4, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

2 issues found across 5 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread index.html Outdated
Comment thread index.html Outdated
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
Signed-off-by: Huỳnh Thương <252359928+Huynhthuongg@users.noreply.github.com>

@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: 6f2d6e922a

ℹ️ 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 (@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 (@codex) address that feedback".

Comment thread index.html
Comment thread index.html

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1 issue found across 2 files (changes from recent commits).

Tip: Review your code locally with the cubic CLI to iterate faster.

Re-trigger cubic

Comment thread scripts/smoke-test-static.mjs
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
Signed-off-by: Huỳnh Thương <252359928+Huynhthuongg@users.noreply.github.com>
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
Signed-off-by: Huỳnh Thương <252359928+Huynhthuongg@users.noreply.github.com>
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
Signed-off-by: Huỳnh Thương <252359928+Huynhthuongg@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working codex documentation Improvements or additions to documentation duplicate This issue or pull request already exists enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed invalid This doesn't seem right question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Xây dựng giao diện Chat AI 3 màn hình

1 participant