Skip to content

feat: Properly Redirect Indie Wikies to specific pages - #19

Merged
kiyarose merged 5 commits into
mainfrom
copilot/feat-preserve-formatting-indep-wikis
Apr 12, 2026
Merged

feat: Properly Redirect Indie Wikies to specific pages#19
kiyarose merged 5 commits into
mainfrom
copilot/feat-preserve-formatting-indep-wikis

Conversation

CopilotAI commented Apr 12, 2026

Copy link
Copy Markdown
Contributor

Redirects to independent wikis always landed on the wiki root instead of the specific article. The IWB data uses plain prefixes (e.g. /wiki/) for destination_content_path no $1 placeholder, so .replace(‘$1’, article was a no-op on every match

// Before — $1 not present, replace() is a no-op → always "/wiki/"destPath=wiki.destination_content_path.replace('$1',encodedArticle)// Afterif(wiki.destination_content_path.includes('$1')){destPath=wiki.destination_content_path.replace('$1',encodedArticle)}else{destPath=wiki.destination_content_path+encodedArticle}

CopilotAI linked an issue Apr 12, 2026 that may be closed by this pull request
@deepsource-io

deepsource-ioBot commented Apr 12, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in e09b771...f78d5a4 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall GradeSecurity

Reliability

Complexity

Hygiene

Code Review Summary

AnalyzerStatusUpdated (UTC)Details
PHPApr 12, 2026 3:51a.m.Review ↗
ShellApr 12, 2026 3:51a.m.Review ↗
JavaScriptApr 12, 2026 3:51a.m.Review ↗
SwiftApr 12, 2026 3:51a.m.Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

CopilotAI changed the title [WIP] Add functionality to preserve formatting on independent wikisfeat: preserve article path when redirecting to independent wikis (v2.2.0)Apr 12, 2026
CopilotAI requested a review from kiyaroseApril 12, 2026 03:32
@kiyarose

Copy link
Copy Markdown
Member

@codex review

@kiyarose

Copy link
Copy Markdown
Member

@codex[agent] review

@Codex

CodexAI commented Apr 12, 2026

Copy link
Copy Markdown

@codex[agent] review

Reviewed the changes—no issues found in the new article path handling or main-page fallback logic; no further edits needed. Tests not run in this pass.

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

@CodexCodexAI changed the title feat: preserve article path when redirecting to independent wikis (v2.2.0)Addressing PR commentsApr 12, 2026
@kiyarose
kiyarose marked this pull request as ready for review April 12, 2026 03:38
CopilotAI review requested due to automatic review settings April 12, 2026 03:38
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Already looking forward to the next diff.

ℹ️ 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".

CopilotAI 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.

Pull request overview

Updates the app/extension version numbers and adjusts wiki destination URL building to better handle main-page redirects and non-templated destination paths.

Changes:

  • Bumped MARKETING_VERSION to 2.2.0 and CURRENT_PROJECT_VERSION to 4 in both macOS and iOS Xcode projects.
  • Bumped extension manifest.json versions to 2.2.0 (macOS + iOS bundles).
  • Enhanced findMatchingWiki() URL construction to (a) map wiki-root / origin-main-page visits to a destination main page when provided, and (b) support destination content paths that are either $1-templated or prefix-style.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
FileDescription
mos/Flean.xcodeproj/project.pbxprojVersion/build number bump for macOS targets.
mos/Flean Extension/Resources/scripts/wiki-data-manager.jsImproves destination URL generation and main-page handling.
mos/Flean Extension/Resources/manifest.jsonExtension version bump to 2.2.0.
ios/Flean.xcodeproj/project.pbxprojVersion/build number bump for iOS targets.
ios/extention/Resources/scripts/wiki-data-manager.jsMirrors the macOS URL-generation improvements for iOS extension bundle.
ios/extention/Resources/manifest.jsonExtension version bump to 2.2.0.

@kiyarosekiyarose changed the title Addressing PR commentsfeat: Properly Redirect Indie Wikies to specific pagesApr 12, 2026
@kiyarose
kiyarose enabled auto-merge April 12, 2026 03:47
…swift
Add Swift CodeQL workflow and fix Dependabot config
@kiyarose
kiyarose merged commit dda83cf into mainApr 12, 2026
8 checks passed
@kiyarose
kiyarose deleted the copilot/feat-preserve-formatting-indep-wikis branch April 12, 2026 04:17
Sign up for freeto 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.

feat: preserve formatting on independent wikis

4 participants

@kiyarose@Codex