どうにかスクリプトの関数を正しく動作させることができました。
Claude Code の助けも借りて、変更点を以下に示します ![]()
=>mongo.rb
def posts(offset = 0, page_size = 1000)
post_keys = mongo.find(_key: "posts:pid").skip(offset).limit(page_size).pluck(:value)
post_keys
.map { |pid| post(pid) }
.compact # <-- nilの結果をすべて削除(孤立したpid)
post_keys.map { |post_key| post(post_key) }
end
def post(id)
post = mongo.find(_key: "post:#{id}").first
return nil if post.nil? # <-- nullチェック
post["timestamp"] = timestamp_to_date(post["timestamp"])
if post["upvoted_by"] = mongo.find(_key: "pid:#{id}:upvote").first
post["upvoted_by"] = mongo.find(_key: "pid:#{id}:upvote").first[:members]
else
post["upvoted_by"] = []
end
post["pid"] = post["pid"].to_s
post["deleted"] = post["deleted"].to_s
post
end
=>nodebb.rb
create_posts(posts, total: post_count, offset: offset) do |post|
# postがnullの場合はスキップ
# merged_postの場合はスキップ
next if post.nil?
next if @merged_posts_map[post["pid"]]
# 削除済みの場合はスキップ
next if post["deleted"] == "1"
raw = post["content"]
post_id = "p#{post["pid"]}"
next if raw.blank?
topic = topic_lookup_from_imported_post_id("t#{post["tid"]}")
unless topic
puts "Topic with id #{post["tid"]} not found, skipping"
next
end
これで正しく動作しているようです。
Discourse の内部アーキテクチャの観点から見て、これがどれほど正しいかはわかりませんが、一見したところ動作しているようです。
改善点や最適化に関する提案はいつでも歓迎します。